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

be more strict on CSR version number allowed #5841

Closed
wants to merge 2 commits into from

Conversation

JacobBarthelmeh
Copy link
Contributor

ZD15260

This impacts CSR creation and no longer allows making a CSR with a version other than 0. The only downside currently is that wc_InitCert defaults to version '2' for the typical X509 creation, which if a user does not explicitly update the default version after calling wc_InitCert they would now see a ASN_VERSION_E when creating CSR's.

@JacobBarthelmeh JacobBarthelmeh self-assigned this Dec 2, 2022
@JacobBarthelmeh
Copy link
Contributor Author

ok to test

@wolfSSL-Bot
Copy link

Can one of the admins verify this patch?

@JacobBarthelmeh
Copy link
Contributor Author

This resolved a

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