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

openssl: auto-detect SSL_R_TLSV13_ALERT_CERTIFICATE_REQUIRED #11617

Closed
wants to merge 1 commit into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Aug 7, 2023

OpenSSL 1.1.1 defines this macro, but no ealier version, or any of the popular forks (yet). Use the macro itself to detect its presence, replacing the hard-wired fork-specific conditions.

This way the feature will enable automatically when forks implement it, while also shorter and possibly requiring less future maintenance.

Follow-up to 94241a9 #6721

Closes #11617

OpenSSL 1.1.1 defines this macro, but no ealier version, or any of
the popular forks (yet). Use the macro itself to detect its presence,
replacing the hard-wired fork-specific conditions.

This way the feature will enable automatically when forks implement it,
while also shorter and possibly requiring less future maintenance.

Follow-up to 94241a9 curl#6721

Closes curl#11617
@vszakats vszakats closed this in 00a70cc Aug 8, 2023
@vszakats vszakats deleted the openssl-autodetect-feat branch August 8, 2023 09:13
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
OpenSSL 1.1.1 defines this macro, but no ealier version, or any of the
popular forks (yet). Use the macro itself to detect its presence,
replacing the hard-wired fork-specific conditions.

This way the feature will enable automatically when forks implement it,
while also shorter and possibly requiring less future maintenance.

Follow-up to 94241a9 curl#6721

Reviewed-by: Jay Satiro
Closes curl#11617
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants