Skip to content

Pick a CA bundle root that has the fields ReadPEMCert checks - #74

Open
quinnj wants to merge 1 commit into
mainfrom
fix/ca-bundle-test
Open

quinnj wants to merge 1 commit into
mainfrom
fix/ca-bundle-test

Conversation

@quinnj

@quinnj quinnj commented Sep 30, 2026

Copy link
Copy Markdown
Member

ReadPEMCert took the second certificate in Mozilla's CA bundle and asserted its subject has an OU. In the bundle that ships with current Julia (1.12+ and nightly), that entry is COMODO ECC Certification Authority, which has no OU. So the test fails on every PR right now (#69, #71, ...).

The test now uses the first root whose subject has an OU. The fix comes from @krynju's #73 (credited as co-author), split out so the smaller PRs can go green without waiting on #73.

🤖 Generated with Claude Code

The test took the second entry of Mozilla's CA bundle and asserted it
has an OU. The bundle shipped with current Julia versions has no OU in
that entry, so the test failed on Julia 1 and nightly for every PR.

Co-authored-by: krynju <25935996+krynju@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant