Skip to content

ceremony: Add support for configuring cross-sign EKUs#8750

Merged
aarongable merged 9 commits into
mainfrom
cross-sign-ekus
May 12, 2026
Merged

ceremony: Add support for configuring cross-sign EKUs#8750
aarongable merged 9 commits into
mainfrom
cross-sign-ekus

Conversation

@aarongable
Copy link
Copy Markdown
Contributor

As of 2025-09-15, the CCADB policy requires that all subordinate CA certificates, including cross-signs of existing roots, must have an EKU extension. As of 2026-06-15, the Chrome policy will require that all new subordinate CA certificates only contain the tlsServerAuth EKU. This change allows us to configure the set of EKUs that will appear in a CA certificate, while restricting that selection per the aforementioned requirements.

@aarongable aarongable requested a review from a team as a code owner May 9, 2026 08:03
@aarongable aarongable requested a review from ezekiel May 9, 2026 08:03
ezekiel
ezekiel previously approved these changes May 9, 2026
@pgporada
Copy link
Copy Markdown
Member

pgporada commented May 9, 2026

The ceremony tool readme requires an update for at least that line.

Comment thread cmd/ceremony/cert.go Outdated
Copy link
Copy Markdown
Contributor

@jsha jsha left a comment

Choose a reason for hiding this comment

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

Looks good with a couple of comment nits. Also: we should add some unittest cases for the new rejections added. And perhaps we can add a check that notBefore is before June 15, 2026 when EKU contains [serverAuth, clientAuth]?

Comment thread cmd/ceremony/cert.go
Comment thread cmd/ceremony/main.go
aarongable and others added 3 commits May 11, 2026 09:44
Co-authored-by: Jacob Hoffman-Andrews <github@hoffman-andrews.com>
@jsha
Copy link
Copy Markdown
Contributor

jsha commented May 11, 2026

I've updated this PR with a unittest for the behavior in general, plus a safety check on the notBefore date relative to Chrome's deadline for non-dedicated cross-signs, and a unittest for the safety check. Ready for re-review.

Copy link
Copy Markdown
Contributor Author

@aarongable aarongable left a comment

Choose a reason for hiding this comment

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

Thanks for the tests, lgtm with one typo:

Comment thread cmd/ceremony/cert.go Outdated
Copy link
Copy Markdown
Contributor

@jsha jsha left a comment

Choose a reason for hiding this comment

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

Additional diff looks good to me.

@jsha jsha requested review from beautifulentropy and ezekiel May 11, 2026 22:14
@aarongable aarongable merged commit c62e796 into main May 12, 2026
17 checks passed
@aarongable aarongable deleted the cross-sign-ekus branch May 12, 2026 00:36
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.

4 participants