pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] rdhabalia opened a new pull request #3501: [pulsar-broker] Make non-tls web/broker-service optional
Date Fri, 01 Feb 2019 02:37:14 GMT
rdhabalia opened a new pull request #3501: [pulsar-broker] Make non-tls web/broker-service
optional
URL: https://github.com/apache/pulsar/pull/3501
 
 
   ### Motivation
   Right now, broker always listens on non-tls webservice(`8080`) and brokerservice(`6650`)
port even though when user doesn't define it. So, broker always listens on non-secure port
by default even when user doesn't want. And making those port optional will be tricky because
broker has direct dependencies of them at many places (eg: while registering load-balancer
node, always selecting broker with http-url,  creating ownership-cache with non-tls url, etc.)
   
   ### Modification
   - Make non-secured port and service-url optional 
   - If non-secure web/broker service url not present then fallback to secured url.
   
   ### Result
   User can start pulsar in a secure mode only.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

Mime
View raw message