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.

Sign up for GitHub

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

Jump to bottom

Add parsing of Name Constraints extension, allow handling raw Other Name #9894

Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Add parsing of Name Constraints extension, allow handling raw Other Name #9894

wants to merge 1 commit into from
+366 −35

Conversation

BrianSipos
Copy link

@BrianSipos BrianSipos commented Jan 10, 2025 •
edited
Loading

Description

This adds capability for mbedtls to handle the Name Constraints extension in accordance with RFC 5280.
This also allows mbedtls to index into and show info for general Other Name forms, and a specific text display for BundleEID Other Name.

PR checklist

This was referenced Jan 10, 2025
Open
Open
@BrianSipos BrianSipos force-pushed the 8759-name-constraints branch from 236b376 to 9425b9e Compare January 10, 2025 21:13
@minosgalanakis minosgalanakis added needs-work needs-review Every commit must be reviewed by at least two team members, component-x509 needs-ci Needs to pass CI tests needs-reviewer This PR needs someone to pick it up for review size-m Estimated task size: medium (~1w) priority-medium Medium priority - this can be reviewed as time permits labels Jan 21, 2025
@minosgalanakis
Copy link
Contributor

Hi and thank you for your contribution. Would it be possible to rebase on top of latest development, and complete the Pull requests description?

That looks like a feature that could be added to 3.6 as well.

@BrianSipos BrianSipos force-pushed the 8759-name-constraints branch from 9425b9e to ce1cf18 Compare January 22, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
component-x509 needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review needs-work priority-medium Medium priority - this can be reviewed as time permits size-m Estimated task size: medium (~1w)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BrianSipos @minosgalanakis