You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2021/03/18 15:31:49 UTC

[GitHub] [ratis-thirdparty] adoroszlai opened a new pull request #14: RATIS-1342. Upgrade Netty to 4.1.60.Final

adoroszlai opened a new pull request #14:
URL: https://github.com/apache/ratis-thirdparty/pull/14


   ## What changes were proposed in this pull request?
   
   Bump Netty version to latest 4.1 release with fix for:
   
    * https://github.com/netty/netty/security/advisories/GHSA-5mcr-gq6c-3hq2
    * https://github.com/netty/netty/security/advisories/GHSA-wm47-8v5p-wjpj
   
   https://issues.apache.org/jira/browse/RATIS-1342
   
   ## How was this patch tested?
   
   Tested locally: built `ratis-thirdparty-misc`, built Ratis with it.  Ran Ratis tests and some Ozone acceptance tests using this custom build.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [ratis-thirdparty] adoroszlai commented on pull request #14: RATIS-1342. Upgrade Netty to 4.1.60.Final

Posted by GitBox <gi...@apache.org>.

adoroszlai commented on pull request #14:
URL: https://github.com/apache/ratis-thirdparty/pull/14#issuecomment-802598788


   Thanks @cnauroth and @risdenk for the review, @szetszwo for merging it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [ratis-thirdparty] szetszwo commented on pull request #14: RATIS-1342. Upgrade Netty to 4.1.60.Final

Posted by GitBox <gi...@apache.org>.

szetszwo commented on pull request #14:
URL: https://github.com/apache/ratis-thirdparty/pull/14#issuecomment-802442789


   Thanks @adoroszlai for working on this.
   
   Thanks also @risdenk and @cnauroth for reviewing the change.
    
   I just have merged this pull request.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [ratis-thirdparty] szetszwo merged pull request #14: RATIS-1342. Upgrade Netty to 4.1.60.Final

Posted by GitBox <gi...@apache.org>.

szetszwo merged pull request #14:
URL: https://github.com/apache/ratis-thirdparty/pull/14


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [ratis-thirdparty] cnauroth commented on pull request #14: RATIS-1342. Upgrade Netty to 4.1.60.Final

Posted by GitBox <gi...@apache.org>.

cnauroth commented on pull request #14:
URL: https://github.com/apache/ratis-thirdparty/pull/14#issuecomment-802135226


   +1 (non-binding)
   
   I checked out the change locally.  The build was successful and all tests passed.
   
   Thank you for the change, @adoroszlai .


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org