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.

Sign up for GitHub

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

Jump to bottom

Support longer certificate chains. #162

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Support longer certificate chains. #162

merged 1 commit into from
Jan 21, 2025

Conversation

anhu
Copy link
Member

@anhu anhu commented Dec 31, 2024 •
edited
Loading

Fixes ZD18976

@anhu anhu self-assigned this Dec 31, 2024
@anhu
Copy link
Member Author

anhu commented Jan 21, 2025

Confirmed this will serve our needs.

Tested with the following:

Build and install wolfssl with: ./configure --enable-altcertchains --enable-wolfclu

Build wolfclu as usual.

./wolfssl verify -partial_chain -CAfile first.pem  second.pem
./wolfssl verify -partial_chain -CAfile second.pem third.pem
./wolfssl verify -partial_chain -CAfile third.pem fourth.pem

@anhu anhu requested a review from JacobBarthelmeh January 21, 2025 04:57
@anhu anhu assigned JacobBarthelmeh and unassigned anhu Jan 21, 2025
@JacobBarthelmeh JacobBarthelmeh merged commit 665adb3 into wolfSSL:main Jan 21, 2025
15 checks passed
@JacobBarthelmeh JacobBarthelmeh mentioned this pull request Jan 22, 2025
Merged
@anhu anhu mentioned this pull request Jan 22, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JacobBarthelmeh JacobBarthelmeh JacobBarthelmeh approved these changes

Assignees

@JacobBarthelmeh JacobBarthelmeh

@wolfSSL-Bot wolfSSL-Bot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@anhu @JacobBarthelmeh @wolfSSL-Bot