jiazhai opened a new pull request #2359: Issue #2304: add encoding for 
WebTargets parts
URL: https://github.com/apache/incubator-pulsar/pull/2359
 
 
   ### Motivation
   
   Pulsar admin commands will fail with some url style parts,
   ```
   root@pulsar-admin:/pulsar# pulsar-admin namespaces grant-permission 
--actions produce,consume --role 'spiffe://developer/passport' 
core-platform/testing
   HTTP 404 Not Found
   Reason: HTTP 404 Not Found
   ```
   This PR try to fix it.
   
   ### Modifications
   
   - use URLEncoder to encode webTarget parts.
   - add tests to verify
   
   ### Result
   
   all UT passed

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


With regards,
Apache Git Services

Reply via email to