Skip to content

Fixes for ASN original #7785

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

Merged
merged 5 commits into from
Jul 26, 2024
Merged

Fixes for ASN original #7785

merged 5 commits into from
Jul 26, 2024

Conversation

dgarske
Copy link
Contributor

@dgarske dgarske commented Jul 24, 2024

Description

Testing

./configure --enable-asn=original --enable-all && make check
./configure --enable-asn=original && make check

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

…riginal (old). Add the new limit checks for alt names and subtree to the old ASN code.
@dgarske dgarske self-assigned this Jul 24, 2024
@dgarske dgarske requested review from bandi13 and ColtonWilley July 24, 2024 15:42
ColtonWilley
ColtonWilley previously approved these changes Jul 24, 2024
…valid OID. Disable invalid UTF8 test for old ASN (only supported with newer ASN template).
bandi13
bandi13 previously approved these changes Jul 24, 2024
Copy link
Contributor

@bandi13 bandi13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bandi13 bandi13 assigned wolfSSL-Bot and unassigned dgarske Jul 24, 2024
@bandi13 bandi13 requested a review from SparkiDev July 24, 2024 21:16
@dgarske dgarske assigned SparkiDev and wolfSSL-Bot and unassigned wolfSSL-Bot Jul 24, 2024
@SparkiDev SparkiDev assigned dgarske and unassigned SparkiDev Jul 24, 2024
@bandi13 bandi13 removed the request for review from SparkiDev July 26, 2024 18:48
@bandi13 bandi13 merged commit b1765ca into wolfSSL:master Jul 26, 2024
121 checks passed
@dgarske dgarske deleted the asn_original branch August 13, 2024 19:24
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.

5 participants