Skip to content

Commit d21ac9a

Browse files
author
Roland Hedberg
committed
Fixed one security bug pointed out by Ehsan Foroughi.
1 parent 6235254 commit d21ac9a

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/saml2/response.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -850,9 +850,13 @@ def verify(self, key_file=""):
850850
"""
851851

852852
try:
853-
self._verify()
854-
except AssertionError:
853+
res = self._verify()
854+
except AssertionError as err:
855+
logger.error("Verification error on the response: %s" % err)
855856
raise
857+
else:
858+
if res is None:
859+
return None
856860

857861
if not isinstance(self.response, samlp.Response):
858862
return self

0 commit comments

Comments
 (0)