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

[CHANGED] Deprecate NATS_BUILD_TLS_USE_OPENSSL_1_1_API CMake variable #617

Merged
merged 1 commit into from
Nov 18, 2022

Conversation

kozlovic
Copy link
Member

Building with TLS will now default to OpenSSL 1.1+/3.0+. The OpenSSL APIs used by the C Client are the same in OpenSSL 3.0 and 1.1, so we did not need to update code to support OpenSSL 3.0. The OpenSSL 1.1+ has an EOL of Sep 2023.

The OpenSSL 1.0.2 APIs being used can be possibly removed in a future release. It will still be possible until then to build the NATS C Client with OpenSSL 1.0.2 by setting NATS_BUILD_TLS_USE_OPENSSL_1_1_API to OFF.

Resolves #614

Signed-off-by: Ivan Kozlovic [email protected]

Building with TLS will now default to OpenSSL 1.1+/3.0+. The OpenSSL
APIs used by the C Client are the same in OpenSSL 3.0 and 1.1, so
we did not need to update code to support OpenSSL 3.0. The OpenSSL
1.1+ has an EOL of Sep 2023.

The OpenSSL 1.0.2 APIs being used can be possibly removed in a future
release. It will still be possible until then to build the NATS C
Client with OpenSSL 1.0.2 by setting NATS_BUILD_TLS_USE_OPENSSL_1_1_API
to OFF.

Resolves #614

Signed-off-by: Ivan Kozlovic <[email protected]>
Copy link
Member

@aricart aricart left a comment

Choose a reason for hiding this comment

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

LGTM!

@kozlovic kozlovic merged commit 2b8543a into dev Nov 18, 2022
@kozlovic kozlovic deleted the deprecate_openssl_pre_1_1_0 branch November 18, 2022 23:03
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.

2 participants