rdhabalia commented on a change in pull request #1002: Making Pulsar Proxy more 
secure
URL: https://github.com/apache/incubator-pulsar/pull/1002#discussion_r161096455
 
 

 ##########
 File path: 
pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/ProxyConfiguration.java
 ##########
 @@ -35,6 +35,14 @@
     // ZooKeeper session timeout
     private int zookeeperSessionTimeoutMs = 30_000;
 
+    // If Discovery Service is Disabled the proxy will just authenticate the 
client 
+    // and forward all requests to a VIP or any other service discovery port 
+    private boolean discoveryServiceEnabled = true;
+    
+    // if Service Discovery is Disabled this url should point to the discovery 
service provider. 
+    private String discoveryServiceURL = "pulsar://localhost:6650/";
 
 Review comment:
   should we rename it to `brokerServiceUrl` as in this case proxy will 
directly try to communicate broker with brokerserviceUrl?

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to