[ 
https://issues.apache.org/jira/browse/SENTRY-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13997718#comment-13997718
 ] 

Lenni Kuff commented on SENTRY-191:
-----------------------------------

Based on our discussion yesterday, it seems the primary goal is to protect rpcs 
that modify policy metadata (grant/revoke statements). Currently all RPCs 
require passing the user credentials. At a minimum I feel we should:
* Remove the need for passing the credentials for all metadata RPCs (list* 
calls)
* Only require passing the username and have the sentry service resolve the 
user -> group mapping.

> Sentry Policy Service should not require passing the RPC requestor's 
> user/group information
> -------------------------------------------------------------------------------------------
>
>                 Key: SENTRY-191
>                 URL: https://issues.apache.org/jira/browse/SENTRY-191
>             Project: Sentry
>          Issue Type: Bug
>    Affects Versions: 1.3.0
>            Reporter: Lenni Kuff
>            Priority: Blocker
>
> Sentry Policy Service should not require passing the RPC requestor's 
> user/group information. Currently this is done to "authorize" whether a user 
> can execute a GRANT/REVOKE statement since only pre-selected set of admin 
> users run grant/revoke statements. This does not seem very secure and also 
> couples "authorization" with the storing of policy metadata.
> I propose that instead of this model, a default "admin" role be introduced. 
> On Sentry Service startup the the role be populated with set of valid admin 
> users as specified in the sentry-service.xml configuration file.
> When GRANT/REVOKE statements are run they should be treated the same as any 
> other SQL statement and authorized at the binding layer (if the give user 
> isn't part of the "admin" role then fail the request). 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to