jinmeiliao commented on a change in pull request #6835:
URL: https://github.com/apache/geode/pull/6835#discussion_r708506757
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/CacheClientUpdater.java
##########
@@ -1763,6 +1767,19 @@ private void processMessages() {
}
}
+ private void handleAuthenticate(Message clientMessage) {
+ // if client is in multi-user mode, the CacheClientUpdator (at this point)
+ // can't differentiate which user this message is intended to. so throw
exception for now
+ // one possible solution is re-authenticate all users in this client
+ if (qManager.getPool().getMultiuserAuthentication()) {
+ throw new UnsupportedOperationException(
+ "Multi-user mode doesn't support re-authentication. This client will
be closed.");
+ }
+ Long userId = AuthenticateUserOp
+ .executeOn(location, qManager.getPool(),
this.system.getSecurityProperties());
Review comment:
I will continue to write more dunit tests in the coming PR.
--
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]
