[GitHub] [pulsar] Technoboy- opened a new pull request, #15551: [fix][broker] Fix REST produce msg redirect issue.

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

Technoboy- opened a new pull request, #15551:
URL: https://github.com/apache/pulsar/pull/15551

   Master Issue: #15546
   
   ### Motivation
   When lookup the topic ownership using REST produce, the redirect URI is incorrect, because :
   
   ```
   uri.getPath(false); //Get the path of the current request relative to the base URI as a string.
   ```
   So the redirect URI does not contain the base path:
   ```
   URI redirectURI = new URI(String.format("%s%s", redirectAddresses.get(0), uri.getPath(false))) 
   ```
   
   ### Documentation
   
   - [x] `no-need-doc` 
   (Please explain why)
   


-- 
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

[GitHub] [pulsar] congbobo184 merged pull request #15551: [fix][broker] Fix REST produce msg redirect issue.

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

congbobo184 merged PR #15551:
URL: https://github.com/apache/pulsar/pull/15551


-- 
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