stoty commented on code in PR #104:
URL:
https://github.com/apache/phoenix-queryserver/pull/104#discussion_r949825552
##########
phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java:
##########
@@ -595,6 +595,15 @@ public <T> T doAsRemoteUser(String remoteUserName, String
remoteAddress,
// Proxy this user on top of the server's user (the real user). Get a
cached instance, the
// LoadingCache will create a new instance for us if one isn't cached.
+
+ // realm got removed from remoteUserName in CALCITE-4152
+ // so we remove the instance name to avoid geting
KerberosName$NoMatchingRule exception
+
+ int separatorIndex = remoteUserName.indexOf('/');
Review Comment:
I think you could use remoteUserName =
KerberosName(remoteUserName).getServiceName() for 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]
