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


   ## What changes were proposed in this pull request?
   
   RATIS-1355 upgraded Netty due to a CVE, but no matching gRPC was available 
at that time.  Now the latest gRPC version officially supports Netty 4.1.63.
   
   https://issues.apache.org/jira/browse/RATIS-1466
   
   ## How was this patch tested?
   
   Built Ratis with the updated thirdparty library, ran tests.
   
   Verified that no native libraries are left unshaded:
   
   ```
   $ find misc/target/classes/META-INF/native -type f -name '*netty*' -not 
-name '*ratis_thirdparty*' | wc -l
          0
   
   $ unzip -t misc/target/ratis-thirdparty-misc-0.8.0-SNAPSHOT.jar 
'META-INF/native/*'
   Archive:  misc/target/ratis-thirdparty-misc-0.8.0-SNAPSHOT.jar
       testing: META-INF/native/         OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_tcnative_osx_x86_64.jnilib 
  OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_tcnative_linux_x86_64.so   
OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_transport_native_epoll_x86_64.so
   OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_transport_native_epoll_aarch_64.so
   OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_tcnative_linux_aarch_64.so 
  OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_transport_native_kqueue_x86_64.jnilib
   OK
       testing: 
META-INF/native/liborg_apache_ratis_thirdparty_netty_resolver_dns_native_macos_x86_64.jnilib
   OK
       testing: 
META-INF/native/org_apache_ratis_thirdparty_netty_tcnative_windows_x86_64.dll   
OK
   ```


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to