Visitar URL original
Add configurable EKU to X.509 verification policies by arpitkjaiswal · Pull Request #15835 · pyca/cryptography · GitHub
Skip to content

Add configurable EKU to X.509 verification policies - #15835

Open
arpitkjaiswal wants to merge 5 commits into
pyca:mainfrom
arpitkjaiswal:codex/configurable-x509-eku
Open

arpitkjaiswal wants to merge 5 commits into
pyca:mainfrom
arpitkjaiswal:codex/configurable-x509-eku

Conversation

@arpitkjaiswal

@arpitkjaiswal arpitkjaiswal commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Addresses the EKU portion of #14765.

Adds PolicyBuilder.extended_key_usage(ObjectIdentifier) so callers can choose the OID checked by X.509 verification. The default clientAuth/serverAuth values remain when the method is omitted. Under the default extension policies, an end-entity certificate may omit its EKU extension; if present, it must include the configured OID. The configured OID is also used by the default CA EKU validator.

Adds coverage for client and server policies, invalid input, and the builder's set-once behavior. Updates the X.509 verification documentation and changelog.

@arpitkjaiswal
arpitkjaiswal force-pushed the codex/configurable-x509-eku branch from 36b208d to 37378e9 Compare October 15, 2026 21:57
@arpitkjaiswal

Copy link
Copy Markdown
Contributor Author

why it's not getting merged?

@alex

alex commented Oct 29, 2026

Copy link
Copy Markdown
Member

While sometimes the answer to your question is "because we lost track of this PR or forgot about it", by an overwhelming margin, the reason something hasn't been reviewed or merged is because the maintainers, who are all volunteers, haven't had time to look at it.

Or more accurately, it's not that we haven't had time, it's that we haven't made time. And asking questions that sound entitled in that way, does nothing to make us want to make time.

This PR will be reviewed eventually, but we don't offer guarantees on when.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants