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

Hadoop QA commented on SENTRY-2231:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12928636/SENTRY-2231.01.patch 
against master.

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} all tests passed

Console output: 
https://builds.apache.org/job/PreCommit-SENTRY-Build/3913/console

This message is automatically generated.

> Fix URI check on List Privileges by Provider in SentryStore
> -----------------------------------------------------------
>
>                 Key: SENTRY-2231
>                 URL: https://issues.apache.org/jira/browse/SENTRY-2231
>             Project: Sentry
>          Issue Type: Bug
>          Components: Sentry
>    Affects Versions: 2.1.0
>            Reporter: Arjun Mishra
>            Assignee: Arjun Mishra
>            Priority: Major
>             Fix For: 2.0.0
>
>         Attachments: SENTRY-2231.01.patch
>
>
> In SentryStore#getMSentryPrivileges when retrieving URI's the query condition 
> "addCustomParam("\"" + authHierarchy.getUri() + "\".startsWith(:URI)", URI, 
> authHierarchy.getUri());", will always be True. This should be changed to 
> filter by appropriate URI.
> For example: when retrieving privileges on URI object "/x/y/z" this condition 
> is checked for: ""file:///x/y/z".startsWith(file:///x/y/z))", which is always 
> true



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to