webpki-ccadb: add V5 certificate records - #130
Conversation
| /// Fetches certificate records from the CCADB All Certificate Records V5 report. | ||
| /// | ||
| /// Records are returned in report order. The report can contain more than one record with the | ||
| /// same certificate fingerprint, so the result is not keyed by fingerprint. |
There was a problem hiding this comment.
IIRC this CSV API is deprecated in favor of a new REST API? Worth checking whether that applies to this?
There was a problem hiding this comment.
Due to limiting the MaxPageSize to 100( See https://github.com/mozilla/CCADB-Tools/blob/master/API_AllCertificateRecords/README.md#pagination-metadata note that CCADB_Settings__c.AllCertRecordsAPI_DefaultPageSize__c is an internal configuration) it seems that using the REST API will require roughly ~103 POSTs instead of a single GET for the full csv
So unless they either improve this or announce the csv as deprecated I think it's better to stick to the CSV and not use the new REST API
What do you prefer that I'd do here?
There was a problem hiding this comment.
The CSV will be deprecated in a few months, so I think we should stick with the REST API.
https://groups.google.com/a/ccadb.org/g/public/c/7TluxT9dtLY/m/Qqj4SiZyBwAJ
There was a problem hiding this comment.
I've started working on this here:
@elichai happy if you want to tag team as I have enough other things on my plate -- feel free to stack PRs on top of mine.
The first thing necessary in order to make
rustls-platform-verifieruse this crate in order to build a list of all CRLsThis uses https://www.ccadb.org/resources "All Certificate Information Reports" V5.
It cannot be merged with the existing endpoint as they return different things where the V5 endpoint can return "duplicate" certificate fingerprints(e.g. one for a root cert and one for an intermediate cert), e.g.:
The plan is to use this in
rustls-platform-verifiertests in order to keep an up-to-date CRL list.Based on: rustls/rustls-platform-verifier#221 (comment)
I will create a follow-up PR in
rustls-platform-verifierin the following days, showing how this is used