[GitHub] massakam opened a new pull request #1862: Fix REST APIs provided by Pulsar proxy

[GitBox <gi...@apache.org>]

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


With regards,
Apache Git Services