Skip to content

Explicitly set the netty SSL Provider to JDK#69

Merged
NikhilCollooru merged 1 commit into
prestodb:masterfrom
NikhilCollooru:fixSSl
Aug 6, 2025
Merged

Explicitly set the netty SSL Provider to JDK#69
NikhilCollooru merged 1 commit into
prestodb:masterfrom
NikhilCollooru:fixSSl

Conversation

@NikhilCollooru
Copy link
Copy Markdown

This is copy of the PR#116 from prestodb/airlift.
prestodb/airlift#116

This code uses the FATAL_ALERT failure behavior, which is only supported by the JDK ssl provider. However, if no SSL Provider is set, it is environment dependent whether the JDK or OpenSSL provider is used. This change Explicitly sets the ssl provider to JDK.

This fixes an issue where servers might fail on start up in ReferenceCountedOpenSslContext with "OpenSSL provider does not support FATAL_ALERT behavior" if they get created with OPEN_SSL as the provider.

This is copy of the PR#116 from prestodb/airlift.
This code uses the FATAL_ALERT failure behavior, which is only
supported by the JDK ssl provider. However, if no SSL Provider
is set, it is environment dependent whether the JDK or OpenSSL
provider is used. This change Explicitly sets the ssl
provider to JDK.

This fixes an issue where servers might fail on start up in ReferenceCountedOpenSslContext with "OpenSSL provider does not support FATAL_ALERT behavior" if they get created with OPEN_SSL as the provider.
@NikhilCollooru NikhilCollooru requested a review from a team as a code owner August 6, 2025 00:02
@NikhilCollooru NikhilCollooru merged commit c0f0952 into prestodb:master Aug 6, 2025
2 checks passed
@NikhilCollooru NikhilCollooru deleted the fixSSl branch August 6, 2025 17:17
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.

3 participants