nodece opened a new pull request, #16650:
URL: https://github.com/apache/pulsar/pull/16650

   Signed-off-by: Zixuan Liu <[email protected]>
   
   Fixes #16626 
   
   ### Motivation
   
   A scenario that running the standalone Pulsar with WebSocket service, then 
using the `token` request param in a WebSocket URL gives an unauthorized 
response, it seems is not working as intended in the 
[documentation](https://pulsar.apache.org/docs/client-libraries-websocket/#query-param-1),
 but it works using a separate WebSocket service.
   
   The root cause is that we did not check whether the token is in the HTTP 
parameters.
   
   ### Modifications
   
   - Add `HttpServletRequestWrapper` to support for get token from HTTP params
   - Add a test for this changes
   
   ### Documentation
     
   - [x] `doc-not-needed` 
   


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

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to