sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Jarcec Cecho (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SENTRY-57) Make tests in TestURIPermissions more restrictive
Date Sat, 10 May 2014 22:11:11 GMT

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

Jarek Jarcec Cecho commented on SENTRY-57:
------------------------------------------

The patch no longer applies cleanly on {{trunk}}. Would you mind rebasing it [~sravya]?

> Make tests in TestURIPermissions more restrictive
> -------------------------------------------------
>
>                 Key: SENTRY-57
>                 URL: https://issues.apache.org/jira/browse/SENTRY-57
>             Project: Sentry
>          Issue Type: Improvement
>            Reporter: Sravya Tirukkovalur
>            Assignee: Sravya Tirukkovalur
>            Priority: Minor
>         Attachments: SENTRY-57.2.patch, SENTRY-57.patch
>
>
> Some minor improvements
> - Fix the role names: for example role "server=server1->db=" + dbName + "->table="
+ tabName + "->action=INSERT" should not be named db1_write, instead it should be "tab1_write"
> - Make the positive tests permissions more restrictive. For example, if a select permission
on URI is required for a command. Make sure to give SELECT instead of ALL.
> - Comment each test with the command and permissions required, so that we can directly
create documentation from here.



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

Mime
View raw message