tngan / samlify

Node.js library for SAML SSO
https://samlify.js.org
MIT License
601 stars 214 forks source link

Failure to verify signature #514

Open varunrau opened 1 year ago

varunrau commented 1 year ago

I'm trying to support SAML for a custom IdP and encountering a signature validation error. After spending some time debugging, I noticed that if I remove this line the signature validation passes. I noticed that other SAML libraries like this one don't remove the signature node.

Would that be safe to do? Why does samlify remove the signature node?