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

About revoked certificates #2

Closed
kelvix opened this issue Jun 12, 2018 · 2 comments
Closed

About revoked certificates #2

kelvix opened this issue Jun 12, 2018 · 2 comments
Assignees

Comments

@kelvix
Copy link

kelvix commented Jun 12, 2018

The readme mentions keeping a database of revoked certificates. Would you instead be interested in a pull request that uses crlDistributionPoints CRL and / or authorityInfoAccess OCSP to determine if a certificate is revoked? These are available as an array of X509::Certificate.extensions and as far as I am aware any issued certificate by a CA will include these extensions on their certificates and could be used to validate a certificate by performing CRL / OCSP request at the time of validation. Some CRLs can get large 10MB+ so OCSP should probably be preferred when the certificate provides authorityInfoAccess.

@jarthod
Copy link
Owner

jarthod commented Jun 13, 2018

When I mean database I mean caching the results from the CRL as it's huge. I'm totally interested in a MR adding OCSP (I had a look at https://github.com/r509/r509 but didn't go much further yet) but for CRL as a fallback I'm not sure as it's super ineffective or requires caching which would mean much more complicated side effects and integrations.
If you're interested in adding both, please add them as options:

  • CRL disabled by default
  • OCSP enabled by default

https://serverfault.com/questions/590504/how-do-i-check-if-my-ssl-certificates-have-been-revoked/732525#732525

https://stackoverflow.com/questions/16244084/how-to-programmatically-check-if-a-certificate-has-been-revoked

@jarthod
Copy link
Owner

jarthod commented Apr 25, 2020

Merged #3 and released 1.3.0

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

No branches or pull requests

3 participants