You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@sentry.apache.org by "Jan Hentschel (JIRA)" <ji...@apache.org> on 2017/04/04 13:51:41 UTC

[jira] [Commented] (SENTRY-378) Clean up the usage of SentryAccessDeniedException

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

Jan Hentschel commented on SENTRY-378:
--------------------------------------

The test failure seem to be unrelated to the actual change. Locally the test passes.

> Clean up the usage of SentryAccessDeniedException
> -------------------------------------------------
>
>                 Key: SENTRY-378
>                 URL: https://issues.apache.org/jira/browse/SENTRY-378
>             Project: Sentry
>          Issue Type: Improvement
>    Affects Versions: 1.4.0
>            Reporter: Sravya Tirukkovalur
>            Assignee: Jan Hentschel
>            Priority: Minor
>             Fix For: 1.8.0
>
>         Attachments: SENTRY-378.001.patch, SENTRY-378.002.patch
>
>
> SentryAccessDeniedException is mostly used when user does is not part of sentry admin group, but is trying to create role, grant role or grant privilege.  Although I see that it is also used else where, there is scope for some clean up. For example:
> SentryStore.getMSentryVersion()



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)