Skip to content

Commit be5d116

Browse files
Merge pull request #560 from erakli/fix_verify_signature
Small fixes in SecurityContext.verify_signature
2 parents 4572ee6 + 9c650e2 commit be5d116

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/saml2/sigver.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1487,10 +1487,10 @@ def _check_signature(self, decoded_xml, item, node_name=NODE_NAME, origdoc=None,
14871487
for cert in cert_from_instance(item)
14881488
]
14891489
else:
1490-
logger.debug('==== Certs from metadata ==== %s: %s ====', issuer, certs)
1490+
logger.debug('==== Certs from metadata ==== %s: %s ====', _issuer, certs)
14911491

14921492
if not certs:
1493-
raise MissingKey(issuer)
1493+
raise MissingKey(_issuer)
14941494

14951495
verified = False
14961496
last_pem_file = None
@@ -1509,9 +1509,6 @@ def _check_signature(self, decoded_xml, item, node_name=NODE_NAME, origdoc=None,
15091509
except XmlsecError as exc:
15101510
logger.error('check_sig: %s', exc)
15111511
pass
1512-
except SignatureError as exc:
1513-
logger.error('check_sig: %s', exc)
1514-
pass
15151512
except Exception as exc:
15161513
logger.error('check_sig: %s', exc)
15171514
raise

0 commit comments

Comments
 (0)