massakam opened a new pull request #1862: Fix REST APIs provided by Pulsar proxy
URL: https://github.com/apache/incubator-pulsar/pull/1862
 
 
   ### Motivation
   
   The following entry points provided by Pulsar proxy does not work properly.
   ```
   /status.html
   /admin/*
   /lookup/*
   ```
   They always return an 404 error.
   
   ### Modifications
   
   Added `clusterName` to the proxy configuration in order to acquire a URL for 
proxying the http requests from configuration store servers.
   The value of the existing `brokerServiceURL` can not be used, because it 
should be a URL that starts with `pulsar://`, not `http://`.
   
   Also, modified WebServer to set the context path when adding proxy servlet.
   Otherwise, the requests to `/status.html` are not handled properly by 
VipStatus.
   
   ### Result
   
   Pulsar proxy will be able to handle the http requests to the above entry 
points successfully.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to