rdhabalia commented on a change in pull request #1002: Making Pulsar Proxy more secure URL: https://github.com/apache/incubator-pulsar/pull/1002#discussion_r161096620
########## File path: pulsar-proxy/src/main/java/org/apache/pulsar/proxy/server/ProxyConnection.java ########## @@ -170,7 +170,11 @@ protected void handleConnect(CommandConnect connect) { // Client is doing a lookup, we can consider the handshake complete and we'll take care of just topics and // partitions metadata lookups state = State.ProxyLookupRequests; - lookupProxyHandler = new LookupProxyHandler(service, this); + if (service.getConfiguration().isDiscoveryServiceEnabled()) { Review comment: is mentioned above, should we derive it based on global-zk value present or not? ---------------------------------------------------------------- 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