Skip to content

Ceremony: add enforcement of acceptable revocation dates and reasons - #9136

Merged
ezekiel merged 2 commits into
mainfrom
crl-profile-ceremony-enforcement
Oct 29, 2026
Merged

ezekiel merged 2 commits into
mainfrom
crl-profile-ceremony-enforcement

Conversation

@aarongable

Copy link
Copy Markdown
Contributor

Add two new checks to our CRL ceremony tool:

  • that the revocation date of each revoked certificate is before the CRL's thisUpdate; and
  • that each revoked certificate has an acceptable revocation reason code.

These checks ensure that our ceremony-produced CRLs comply with the profiles in Section 7.2 of v6.3 of our CP/CPS (https://letsencrypt.org/documents/isrg-cp-cps-v6.3/#7.2-crl-profile). They are belt-and-suspenders checks; they are also checked by the lints in #9129.

Also, update both the crlCeremony and generateCRL functions to build their respective RevocationListEntry and RevocationList objects once all values are prepared, rather than constructing them halfway and then assigning to additional fields afterwards.

@aarongable
aarongable requested a review from a team as a code owner October 29, 2026 19:32
@aarongable
aarongable requested a review from ezekiel October 29, 2026 19:32
ezekiel
ezekiel previously approved these changes Oct 29, 2026
@ezekiel
ezekiel requested review from a team and beautifulentropy and removed request for a team and beautifulentropy October 29, 2026 19:49
@beautifulentropy
beautifulentropy self-requested a review October 29, 2026 20:02

@beautifulentropy beautifulentropy left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I'm certain that this works. Mostly I'd like to see some unit tests. I have provided an idea for an alternative implementation that would be easier to test and a test.

Comment thread cmd/ceremony/main.go Resolved
Comment thread cmd/ceremony/main.go Resolved
@ezekiel
ezekiel merged commit ae35aac into main Oct 29, 2026
19 checks passed
@ezekiel
ezekiel deleted the crl-profile-ceremony-enforcement branch October 29, 2026 21:34
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

Successfully merging this pull request may close these issues.

3 participants