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.

Sign up for GitHub

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

Jump to bottom

Bump netty version to 4.1.75.Final, grpc to 1.45.1 #3163

Merged
merged 1 commit into from Apr 3, 2022
Merged

Bump netty version to 4.1.75.Final, grpc to 1.45.1 #3163

merged 1 commit into from Apr 3, 2022

Conversation

shoothzj
Copy link
Member

@shoothzj shoothzj commented Mar 31, 2022 •
edited

Motivation

original PR #3091
Changelog: https://netty.io/news/2022/03/10/4-1-75-Final.html

Netty 4.1.75 has netty/netty#12103 feature that #2989 needs

Netty false positive netty/netty-tcnative#710 jeremylong/DependencyCheck#4154

Modifications

  • Upgrade Netty from 4.1.74.Final to 4.1.75.Final
  • Netty 4.1.75.Final depends on netty-tc-native 2.0.50, also updates
  • Netty 4.1.75.Final needs grpc version to 1.45.1 see netty: Support pseudo headers in all GrpcHttp2RequestHeaders methods grpc/grpc-java#9004
  • grpc depends on protobuf 3.19.2, also updates
  • grpc depends on gson 2.8.9, update license file
  • grpc depends on http-client-google-http-client 1.41.0, update license file
  • grpc depends on auto.value-auto-value-annotations 1.9.0, update license file

@shoothzj shoothzj mentioned this pull request Mar 31, 2022
Closed
@shoothzj shoothzj force-pushed the netty-4-1-75 branch 3 times, most recently from 4066084 to d6f51ee Compare April 1, 2022 01:54
dlg99
dlg99 approved these changes Apr 1, 2022
View reviewed changes
Copy link
Contributor

@dlg99 dlg99 left a comment

Choose a reason for hiding this comment

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

Great work, thank you!

nicoloboschi
nicoloboschi approved these changes Apr 1, 2022
View reviewed changes
Copy link
Contributor

@nicoloboschi nicoloboschi left a comment

Choose a reason for hiding this comment

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

nice work!

@eolivelli eolivelli merged commit 5502b40 into apache:master Apr 3, 2022
@shoothzj shoothzj deleted the netty-4-1-75 branch April 3, 2022 08:32
@shoothzj shoothzj mentioned this pull request Apr 19, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dlg99 dlg99 dlg99 approved these changes

@nicoloboschi nicoloboschi nicoloboschi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@shoothzj @dlg99 @nicoloboschi @eolivelli