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

Append signature after a specific node #137

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

seorefbe
Copy link

Hello,

To be compliant with urn:oasis:names:tc:SAML:2.0:protocol in a RequestedAuthnContext, the signature have to be append just after the Issuer node.

I need the possibility to define a position in function appendSignature.

Have a good day,

Sébastien

@robrichards
Copy link
Owner

This would be better off passing a reference node rather than a position. I would expect a developer to have access to the node it would be inserted before or after rather than knowing what position it lives within the subtree.

@hebertreis
Copy link

Libera isso aiiii

@hebertreis
Copy link

Please accept this pull request!

@falinhares
Copy link

Was this pull request accepted?

@seorefbe
Copy link
Author

seorefbe commented Sep 6, 2023

Was this pull request accepted?

nop :(

@tvdijen
Copy link
Contributor

tvdijen commented Sep 6, 2023

@seorefbe Perhaps for your SAML2 use-case you can use https://github.com/simplesamlphp/saml2
It's a low-level saml2-library that does most of the heavy lifting for you regarding singing/encryption.

@falinhares
Copy link

I have a tricky SOAP xml that I have to sign just after a element. Any thoughts on how can I do this?

@tvdijen
Copy link
Contributor

tvdijen commented Sep 6, 2023

@robrichards
Copy link
Owner

@seorefbe Did you see my comment about passing in a reference node and not position? If you modify the code to pass in a reference node rather than position number I would add it in.

@falinhares
Copy link

@robrichards I've found a much better solution: so submit to your library only the node that needs the signature and it worked perfectly. Thanks!

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.

5 participants