dontpanico / jwtxploiter Goto Github PK

Describe the bug
Running --find-key-from-jwks raise AttributeError if jwks file does not contain a key that can verify the token

To Reproduce

1. Get a token
2. Get a JWKS file that does not contain the key that can verify the token signature
3. run jwtxpl <token> --find-key-from-jwks <jwks_file>

Expected behavior
No AttributeError should be raised

Additional context
This is due to a typo an attribute has been typed incorrectly), and not for an unhandled exception

I noticed that sometimes install.sh miss to call pip3 install -r requirements.txt. So if when you run the main script you get a depndency-related error, do that manually.

N.B. THIS ISSUE HAS BEEN CLOSED! THE EXCEPTION HAS BEEN HANDLED!

• Before time of writing, FileNotFoundError generated from non existing files passed to -k/--key, was handled only with HS*
  algorithms.

• Now, after last commit, the exception has been handled, no matter which alg has been passed.

Describe the bug
Exception FileNotFoundError not properly handled, when passing a non existing key file to -k/--key.
This happen only with asymmetric algorithms, since it's been handled with HS*

To Reproduce

1. Get a valid JWT (you can grab it from jwt.io)
2. Run an attack using an asymmetric algorithm and passing a non existing key file to -k

   jwtxpl <token> -a rs256 -p <key>:<value> -k /path/to/non/existing

Expected behavior
The script should properly handle this exception and return a custom error

Describe the bug
Jku and x5u attacks that return jwks files miss to generates it.

To Reproduce
Run a jku/x5u attack against a valid target (not --jku-inbody and --x5u-inbody, since they does not returns jwks files)

Expected behavior
The tool should return the crafted jwks file

• Before time of writing, selecting alg none will raise an exceptions. This is due since during validation the script converts algorithms
  to uppercase, but the method which select the signature only look for 'none' or 'None'.

• SOLVED, converting algorithm to uppercase only if its lowercase differ from 'none'.

• I WILL LEAVE THE ISSUE OPENED 3 DAYS IN ORDER TO MAKE IT MORE VISIBLE TO PEOPLE RAN INTO THAT ERROR.