You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ratis.apache.org by GitBox <gi...@apache.org> on 2022/08/07 14:12:29 UTC

[GitHub] [ratis] SzyWilliam opened a new pull request, #707: RATIS-1661. Support configurable hostname in GrpcService

SzyWilliam opened a new pull request, #707:
URL: https://github.com/apache/ratis/pull/707

   ## What changes were proposed in this pull request?
   GrpcService will use `NettyServerBuilder.forport` to bind the listening address, which is by default 0.0.0.0::port. This may bring potential security risks.
   In this PR I propose to use a configurable hostname to construct the binding address. 
   BTW, the InetSocketAddress support ipv6 address, so this PR can also enable users to assign ipv6 address to grpc listening address.
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/RATIS-1661
   
   ## How was this patch tested?
   unit tests.
   


-- 
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: issues-unsubscribe@ratis.apache.org

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

[GitHub] [ratis] SzyWilliam commented on pull request #707: RATIS-1661. Support configurable hostname in GrpcService

Posted by GitBox <gi...@apache.org>.
SzyWilliam commented on PR #707:
URL: https://github.com/apache/ratis/pull/707#issuecomment-1207571818

   @szetszwo Please take a look at this, thanks!


-- 
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: issues-unsubscribe@ratis.apache.org

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

[GitHub] [ratis] szetszwo merged pull request #707: RATIS-1661. Support configurable hostname in GrpcService

Posted by GitBox <gi...@apache.org>.
szetszwo merged PR #707:
URL: https://github.com/apache/ratis/pull/707


-- 
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: issues-unsubscribe@ratis.apache.org

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