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

Update Netty to the latest version #10186

Merged
merged 3 commits into from
Mar 9, 2021
Merged

Update Netty to the latest version #10186

merged 3 commits into from
Mar 9, 2021

Conversation

danotorrey
Copy link
Contributor

Update Netty dependency to the latest version.

This was recently done in Graylog2/gelfclient#52, so the same update is being done in the graylog2-server core while we're looking at this.

@danotorrey danotorrey marked this pull request as ready for review March 2, 2021 16:05
@danotorrey
Copy link
Contributor Author

A quick smoke-test of the application was successful.

@danotorrey danotorrey requested a review from bernd March 2, 2021 16:06
pom.xml Outdated
@@ -135,7 +135,7 @@
<mongodb-driver.version>3.12.1</mongodb-driver.version>
<mongojack.version>2.10.1</mongojack.version>
<natty.version>0.13</natty.version>
<netty.version>4.1.52.Final</netty.version>
<netty.version>4.1.59.Final</netty.version>
<netty-tcnative-boringssl-static.version>2.0.34.Final</netty-tcnative-boringssl-static.version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Would you mind als updating netty-tcnative to 2.0.36?
We've had problems with that in the past, so it's a good practice to update them together

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the suggestion. Done.

@danotorrey danotorrey requested a review from mpfz0r March 9, 2021 08:59
Copy link
Member

@bernd bernd left a comment

Choose a reason for hiding this comment

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

Netty 4.1.60 just got released including a fix for a possible security problem.

See: https://netty.io/news/2021/03/09/4-1-60-Final.html

Copy link
Contributor

@mpfz0r mpfz0r left a comment

Choose a reason for hiding this comment

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

Works for me. Tested HTTPS and TLS enabled beats input

@mpfz0r
Copy link
Contributor

mpfz0r commented Mar 9, 2021

@bernd can you approve or remove your change request?

@bernd bernd requested review from bernd and removed request for bernd March 9, 2021 16:32
@bernd bernd merged commit 47e50ff into master Mar 9, 2021
@bernd bernd deleted the update-netty branch March 9, 2021 16:34
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.

3 participants