You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/01/18 13:04:12 UTC

[GitHub] [pulsar] zymap commented on pull request #9064: Add refresh authentication command in broker

zymap commented on pull request #9064:
URL: https://github.com/apache/pulsar/pull/9064#issuecomment-762237821


   @sijie sorry for the delay.  This PR's goal is to address the issue that pulsar will disconnect the client if the token expired.
   Currently, if the token expired the broker will disconnect the client and make it reconnect to do the authentication again. Because we get the authentication data when a new connection opened and it will not change until it reconnected.
   We handle the refresh command to let the client can replace the token but doesn't need to disconnect from the broker. The client-side still get a token from the method `getAuthData` which is abstracted in the `AuthenticationDataProvider`, user can handle the token generates logic at the method `getAuthData`. 
   /cc @jiazhai Correct me if I missed something.
   
   I improve the expiration token test to include my case.


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

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