michaeljmarshall opened a new pull request, #19283: URL: https://github.com/apache/pulsar/pull/19283
PIP 97: https://github.com/apache/pulsar/issues/12105 ### Motivation While working on PIP 97, I noticed that we do not have a clear contract for the `AuthenticationState` interface. We use the interface in one way in the `ServerCnx` class, but some of our official implementations do not align with the usage and the API. The PR itself provides my proposed authoritative documentation. When we agree on the content of this PR, I will follow up with some changes to the `TokenAuthenticationState` class and potentially others. ### Modifications * Add documentation on how the `AuthenticationState` interface will be used by Pulsar. ### Verifying this change I have studied the code closely to understand how it works, and more importantly, how it should work. ### Does this pull request potentially affect one of the following parts: This change could affect how plugins interact with the `AuthenticationState` object. ### Documentation - [x] `doc` This is a documentation change. -- 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-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org