Add Try/Catch for xml-crypto

krakenjs / passport-saml-encrypted

A strategy for Passport authentication that supports encrypted SAML responses

MIT License

14 stars 26 forks source link

Closed jonfairbanks closed 8 years ago

jonfairbanks commented 8 years ago

Thanks for reviewing. I have merged in changes for each of the feedback items.

Feel free to let me know if additional updates are needed.

jonfairbanks commented 8 years ago

Thanks again for the review. I've updated the if() at line 191 and re-verified that I am using == where appropriate.

lmarkus commented 8 years ago

LGTM! Thanks for the contribution.