Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix handling InclusiveNamespaces in digest calculation #25

Merged
merged 2 commits into from
Jul 8, 2024

Conversation

route443
Copy link
Contributor

@route443 route443 commented Jul 6, 2024

The issue was caused by the incorrect handling of the InclusiveNamespaces element in the digest calculation process. According to the XML-DSig, the exclusive canonicalization algorithm (XML-EXC-C14N) allows for canonicalizing XML without including predefined namespaces unless specified through the PrefixList attribute.

Address issue #23

route443 added 2 commits July 5, 2024 23:16
The issue was caused by the incorrect handling of the InclusiveNamespaces
element in the digest calculation process. According to the XML-DSig,
the exclusive canonicalization algorithm (XML-EXC-C14N) allows for
canonicalizing XML without including predefined namespaces unless specified
through the PrefixList attribute.
Enhanced existing test cases to include InclusiveNamespaces handling in
the signature template. Each test case that verifies the signature now includes
a Transform element with InclusiveNamespaces and a PrefixList attribute.
@route443 route443 merged commit 3de8544 into main Jul 8, 2024
1 check passed
@route443 route443 deleted the fix-saml-digest branch July 8, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant