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

fix(#1210): fix impl of SubjectDirectoryAttributes #1211

Closed
wants to merge 1 commit into from
Closed

fix(#1210): fix impl of SubjectDirectoryAttributes #1211

wants to merge 1 commit into from

Conversation

Taowyoo
Copy link
Contributor

@Taowyoo Taowyoo commented Aug 30, 2023

This PR fixes #1210

@tarcieri
Copy link
Member

Though this does appear to be a bug, it's also a breaking change to incorporate the fix.

We can merge it as part of the next development cycle when we're making breaking changes.

@Taowyoo
Copy link
Contributor Author

Taowyoo commented Aug 30, 2023

sounds good

@baloo baloo added the breaking Change to be merged with next release cycle label Nov 24, 2023
@Taowyoo Taowyoo closed this by deleting the head repository Dec 21, 2023
@tarcieri
Copy link
Member

@Taowyoo We’re still interested in this, FYI

@Taowyoo
Copy link
Contributor Author

Taowyoo commented Dec 21, 2023

Hi @tarcieri, thank you for reminding me. I accidentally close this PR when cleaning repositories. I will create another one with same commit soon.

@Taowyoo
Copy link
Contributor Author

Taowyoo commented Dec 21, 2023

^^^
#1286

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Change to be merged with next release cycle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implementation of x509_cert::ext::pkix::SubjectDirectoryAttributes is wrong/inaccurate
3 participants