[
https://issues.apache.org/jira/browse/PHOENIX-5905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17123665#comment-17123665
]
Istvan Toth commented on PHOENIX-5905:
--------------------------------------
[~rajeshbabu]
Do we need to reset the RpcContext in getUserPermissions() as well as
getUserDefinedPermissions() ?
Your explanation seems to indicate that the reset in getUserPermissions() is
superfluous.
Also, please upload the patch in a format that the pre-commit script can
process and test.
Those are my only concerns, otherwise +1
> Reset user to hbase by changing rpc context before getting user permissions
> on access controller service
> ---------------------------------------------------------------------------------------------------------
>
> Key: PHOENIX-5905
> URL: https://issues.apache.org/jira/browse/PHOENIX-5905
> Project: Phoenix
> Issue Type: Bug
> Reporter: Rajeshbabu Chintaguntla
> Assignee: Rajeshbabu Chintaguntla
> Priority: Major
> Fix For: 5.1.0, 4.16.0
>
> Attachments: PHOENIX-5905.patch, PHOENIX-5905_addendum.patch,
> PHOENIX-5905_addendum2.patch
>
>
> Currently we are calling getUserPermissions with hbase user directly on
> access controller service which is not a rpc call. If we don't reset user
> system user will be considered and might expect extra privileges to return
> the user permissions.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)