[GitHub] [pulsar] wangjialing218 commented on pull request #12072: [Issue 12040][web] Topic lookup with listener header

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

wangjialing218 commented on pull request #12072:
URL: https://github.com/apache/pulsar/pull/12072#issuecomment-921663472


   I think it's not enough to only add the listener head to `lookupTopicAsync`.
   For web request, there is some admin operation which may return a HTTP REDIRECT response with the http url address of the broker own the target topic. For example send a create subscription request to a broker which is not the owner of topic.
   In this case, REDIRECT response using the internal webSeviceUrl and the client which send the HTTP requst may not able to connect to the REDIRECT address from outside.
   Current there is only `brokerSeviceUrl` in advertised address, To support advertised listener for HTTP, we need add `webSeviceUrl` to advertised address first.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org