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

Substitute netty-tcnative-boringssl-static 2.0.17.Final with netty-common and netty-buffer 4.1.50.Final #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leapsky
Copy link
Contributor

@leapsky leapsky commented May 21, 2021

Update netty dependencies in pravega POM: netty-common and netty-buffer

@RaulGracia
Copy link
Contributor

@leapsky why do we need this PR?

@leapsky
Copy link
Contributor Author

leapsky commented May 31, 2021

This issue is related to pravega/pravega#5589

@RaulGracia
Copy link
Contributor

@plipsky what is the state of this PR? can you post here how it has been validated?

@RaulGracia RaulGracia self-requested a review June 23, 2021 09:04
@leapsky
Copy link
Contributor Author

leapsky commented Jun 23, 2021

The PR has been validated running Pravega openmessaging-benchmark tests with TLS connection.

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