You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@sentry.apache.org by "Alexander Kolbasov (JIRA)" <ji...@apache.org> on 2016/11/02 01:02:36 UTC

[jira] [Created] (SENTRY-1515) Cleanup exception handling in SentryStore

Alexander Kolbasov created SENTRY-1515:
------------------------------------------

             Summary: Cleanup exception handling in SentryStore
                 Key: SENTRY-1515
                 URL: https://issues.apache.org/jira/browse/SENTRY-1515
             Project: Sentry
          Issue Type: Bug
          Components: Sentry
    Affects Versions: sentry-ha-redesign
            Reporter: Alexander Kolbasov


The changes to SENTRY-1422 and SENTRY-1512 changed the semantics of several API calls:

    - hasAnyServerPrivileges
    - getMSentryPrivileges
    - getMSentryPrivilegesByAuth
    - getRoleNamesForGroups
    - retrieveFullPrivilegeImage
    - retrieveFullRoleImage
    - retrieveFullPathsImage
    - getAllRoleNames

Previously they were not marked as throwing Exception, but they still could do it. With the change they now ignore exceptions and just log them which may not be the right thing to do. 

Instead they should be marked as throwing exceptions which has consequence for broader APIs which should be marked as well.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)