Skip to content

Commit 7a932c5

Browse files
committed
chore: JWT warnings on failed JWT ops
1 parent ec79b98 commit 7a932c5

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

src/cryptojwt/jws/jws.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -227,13 +227,13 @@ def verify_compact_verbose(self, jws=None, keys=None, allow_none=False, sigalg=N
227227

228228
try:
229229
if not verifier.verify(jwt.sign_input(), jwt.signature(), _key):
230-
continue
231-
except (BadSignature, IndexError):
232-
pass
230+
- continue
231+
except (BadSignature, IndexError) as err:
232+
logger.warning(f'BadSignature caught with {jwt}: "{err}"')
233233
except (ValueError, TypeError) as err:
234-
logger.warning('Exception "{}" caught'.format(err))
234+
logger.warning(f'Exception with {jwt.headers}: "{err}"')
235235
else:
236-
logger.debug("Verified message using key with kid=%s" % key.kid)
236+
logger.debug(f"Verified message using key with kid={key.kid}")
237237
self.msg = jwt.payload()
238238
self.key = key
239239
self._protected_headers = jwt.headers.copy()

src/cryptojwt/jwt.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -372,11 +372,11 @@ def unpack(self, token):
372372
if self.msg_cls:
373373
_msg_cls = self.msg_cls
374374
else:
375-
try:
376-
# try to find a issuer specific message class
377-
_msg_cls = self.iss2msg_cls[_info["iss"]]
378-
except KeyError:
379-
_msg_cls = None
375+
_msg_cls = self.iss2msg_cls.get(_info["iss"], None)
376+
if not _msg_cls:
377+
LOGGER.debug(
378+
f"both msg_cls and iss2msg are None for the issuer {_info['iss']}"
379+
)
380380

381381
if _msg_cls:
382382
vp_args = {"skew": self.skew}

0 commit comments

Comments
 (0)