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.

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

Id cert basic ca path len #241

Merged
merged 9 commits into from
Nov 28, 2022
Merged

Id cert basic ca path len #241

merged 9 commits into from
Nov 28, 2022

Conversation

timbru
Copy link
Contributor

@timbru timbru commented Nov 23, 2022

This caused an interop issue because it's not explicitly forbidden to use this in the context of RFC 6492, 8181 and 8183. Even if this is not normally expected in RPKI resource certificates.

@timbru timbru requested a review from partim November 23, 2022 15:31
@timbru
Copy link
Contributor Author

timbru commented Nov 23, 2022

The build was failing because of clippy warnings. Mostly in related, but unchanged, code. Should I fix those warnings in this PR?

@partim partim merged commit e21a41c into main Nov 28, 2022
@partim partim deleted the id-cert-basic-ca-path-len branch November 28, 2022 09:46
partim added a commit that referenced this pull request Jan 16, 2023
Bug Fixes

* RTR server: Report the right type of query in `Socket::update`. ([#239])
* Allow `pathLenConstraint` in Basic Constraints extension in ID
  certificates. ([#241])
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.

2 participants