[
https://issues.apache.org/jira/browse/PHOENIX-6750?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17581376#comment-17581376
]
ASF GitHub Bot commented on PHOENIX-6750:
-----------------------------------------
joshelser commented on code in PR #104:
URL:
https://github.com/apache/phoenix-queryserver/pull/104#discussion_r949204418
##########
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 don't recall anymore if there's a chance that `remoteUserName` might also
contain an `@REALM`. You could be defensive and guard against that for "free"
(do a second `indexOf` and `substring` block)
> Bump Avatica version to 1.21.0 in queryserver
> ---------------------------------------------
>
> Key: PHOENIX-6750
> URL: https://issues.apache.org/jira/browse/PHOENIX-6750
> Project: Phoenix
> Issue Type: Improvement
> Components: queryserver
> Affects Versions: queryserver-6.0.0
> Reporter: Richárd Antal
> Assignee: Richárd Antal
> Priority: Major
>
> Avatica 1.21 has been released.
> It has several fixes that we want.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
