nodece commented on PR #23905: URL: https://github.com/apache/pulsar/pull/23905#issuecomment-2620383591
> This would be a major change to the admin client. Should be client, not admin client? > It feels that there's something wrong in the > AuthenticationSasl if it gets coupled to the client type. This authentication process requires the broker endpoint, and we pass the URL string, which indicates to the broker which endpoint is being requested. > > Updated pulsar-client calls to utilize the refactored HTTP client and maintain the TLS context seamlessly. > > What does "maintain the TLS context" mean in practice? Just pass the WebTarget to the Authtication. I will make a new PR to handle this thing. > What are the different implementation options? I think we can only pass the WebTarget to the auth. Do you any ideas? Using jakarta.ws.rs instead of javax.ws.rs is a good idea, but fixing the bug is still important, so I want to keep this path. -- 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]
