Skip to content

Update Drift and Netty dependencies to support TLS 1.3 #19782

Merged
ajaygeorge merged 1 commit intoprestodb:masterfrom
sotodel:master
Jun 15, 2023
Merged

Update Drift and Netty dependencies to support TLS 1.3 #19782
ajaygeorge merged 1 commit intoprestodb:masterfrom
sotodel:master

Conversation

@sotodel
Copy link
Contributor

@sotodel sotodel commented Jun 2, 2023

The key change is the version update in dep.drift.version. The version includes a more recent version of Netty, which adds TLS 1.3 support by default (among other things). This will allow Drift clients and servers to establish and accept TLS 1.3 connections where they wouldn't before.

== RELEASE NOTES ==

General Changes

  • Add support for TLS 1.3 in Drift

@sotodel sotodel marked this pull request as ready for review June 2, 2023 18:49
@sotodel sotodel requested a review from a team as a code owner June 2, 2023 18:49
@sotodel sotodel requested a review from ajaygeorge June 2, 2023 18:49
@sotodel sotodel force-pushed the master branch 5 times, most recently from af31919 to d0098b8 Compare June 8, 2023 17:52
@ajaygeorge
Copy link
Contributor

@sotodel
Let us reword the commit message a bit to make it more clear

Update Drift and Netty dependencies to support TLS 1.3
This commit updates Drift version to 1.35 and Netty version 
to 4.1.92.Final to add support for TLS 1.3.

Copy link
Contributor

@ajaygeorge ajaygeorge left a comment

Choose a reason for hiding this comment

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

LGTM % commit message

@sotodel sotodel force-pushed the master branch 4 times, most recently from dd279af to d9ab010 Compare June 14, 2023 22:22
This commit updates Drift version to 1.36 and Netty version
to 4.1.92.Final to add support for TLS 1.3.
@ajaygeorge ajaygeorge changed the title Update Drift dependency to include more recent version of Netty (for TLS 1.3 support) Update Drift and Netty dependencies to support TLS 1.3 Jun 15, 2023
@ajaygeorge ajaygeorge merged commit a433154 into prestodb:master Jun 15, 2023
@wanglinsong wanglinsong mentioned this pull request Jul 27, 2023
28 tasks
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