[
https://issues.apache.org/jira/browse/SENTRY-305?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14036445#comment-14036445
]
Sravya Tirukkovalur commented on SENTRY-305:
--------------------------------------------
[~prasadm], thanks for the patch. It mostly looks good to me. Although I see
two test failures:
Results :
Failed tests:
TestDatabaseProvider.testSetRole:1571 User admin1 shouldn't be able to set
testRole2
Tests in error:
TestDatabaseProvider.testShowAllCurrentRoles:1546 ยป SQL Error while
processing...
Can you take a look and update the patch please?
> SHOW CURRENT ROLES shouldn't require admin privileges
> -----------------------------------------------------
>
> Key: SENTRY-305
> URL: https://issues.apache.org/jira/browse/SENTRY-305
> Project: Sentry
> Issue Type: Bug
> Reporter: Prasad Mujumdar
> Assignee: Prasad Mujumdar
> Attachments: SENTRY-305.4.patch, SENTRY-305.6.patch
>
>
> SHOW CURRENT ROLES shouldn't require admin privileges. If there's an active
> roleset is set, then it does show that information. On the other hand, if
> there's no active roles set then it it requires admin privilege.
> We should fix the inconsistency and allow clients to see current roles
> without any admin privileges.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
