Skip to content

Commit b925bae

Browse files
moonfriendc00kiemon5ter
authored andcommitted
With encryption The ID of each Signature element must be unique
If assertion is gonna be encrypted and the response and the assertion are signed, both signatures will have an ID of `signature1` which leads to an invalid xml because the ids have to be unique. A similar issue has been reported and fixed the same way in server.py with this commit: 4375361
1 parent 67ee332 commit b925bae

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/saml2/entity.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -874,7 +874,7 @@ def _response(
874874
_assertion.signature = pre_signature_part(
875875
_assertion.id,
876876
self.sec.my_cert,
877-
1,
877+
2,
878878
sign_alg=sign_alg,
879879
digest_alg=digest_alg,
880880
)

0 commit comments

Comments
 (0)