jinmeiliao opened a new pull request #6804: URL: https://github.com/apache/geode/pull/6804
…tributes from the server. This logout the given subject when an AuthExpiredException is encountered. This PR doesn't try to clean out the `ClientUserAuths.uniqueIdVsSubject` map for the following reasons: 1. there is no clean way to get access to it unless using the static variable in `ServerConnection` 2. when a clients connects, multiple threads calls into the server to authenticate, resulting into multiple in that entries in that map, each with different unique id and unique subject/principal, even though it's the same user. Logging out one subject nulls one but didn't affect the others. This map will eventually get cleaned up when connection closes or dies, so there isn't much point trying to clean up the nullified subject in this map (as far I can tell). Would love to get more feedback on this. -- 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]
