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

Remove support for certificate validation by fingerprint #431

Closed
thijskh opened this issue Aug 5, 2016 · 1 comment
Closed

Remove support for certificate validation by fingerprint #431

thijskh opened this issue Aug 5, 2016 · 1 comment

Comments

@thijskh
Copy link
Member

@thijskh thijskh commented Aug 5, 2016

simpleSAMLphp allows one to configure trusted certificates either in full, or with just the fingerprint. These two ways of working complicate the code, documentation etc significantly, while the advantage of using fingerprints seems marginal. We should therefore transition to only accept full certificates.

When this is completed the code can also be removed from the saml2 library: simplesamlphp/saml2#68.

Discussion at https://groups.google.com/forum/#!topic/simplesamlphp-dev/MTbTGxAWniw

@tvdijen
Copy link
Member

@tvdijen tvdijen commented Dec 11, 2018

Work on the SAML2 lib has been done.. Will be released in 4.0

@tvdijen tvdijen closed this in 2c8aa8a Feb 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants