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

TLS1_3_VERSION must be ifdef'ed #69

Closed
wants to merge 1 commit into from
Closed

Conversation

lindig
Copy link

@lindig lindig commented Feb 16, 2021

Symbol TLS1_3_VERSION is not available in all openssl releases and
therefore must be ifdef'ed to avoid a compilation error.

This is just a quick fix. I believe the correct thing to do is to use HAVE_TLS13 et al. to guard each of the cases in this match.

Signed-off-by: Christian Lindig [email protected]

@smorimoto
Copy link

#63 has a same change as well.

Symbol TLS1_3_VERSION is not available in all openssl releases and
therefore must be ifdef'ed to avoid compilation error.

Signed-off-by: Christian Lindig <[email protected]>
@lindig
Copy link
Author

lindig commented Feb 16, 2021

That fix is actually more comprehensive - so we can disregard my pull request.

@psafont
Copy link
Contributor

psafont commented Nov 12, 2021

This change was merged in #73

@smimram smimram closed this Nov 15, 2021
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.

4 participants