Skip to content

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