[ https://issues.apache.org/jira/browse/HBASE-7658?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13562033#comment-13562033 ]
Andrew Purtell commented on HBASE-7658: --------------------------------------- bq. I think we should throw an exception instead of just logging. No reason not to. See also HBASE-7333 for a larger discussion on shell support for security, including the 'grant' command. > grant with an empty string as permission should throw an exception > ------------------------------------------------------------------ > > Key: HBASE-7658 > URL: https://issues.apache.org/jira/browse/HBASE-7658 > Project: HBase > Issue Type: Bug > Components: security > Affects Versions: 0.96.0, 0.94.4 > Reporter: Matteo Bertozzi > Assignee: Matteo Bertozzi > Priority: Trivial > > If someone specify an empty permission > {code}grant 'user', ''{code} > AccessControlLists.addUserPermission() output a log message and doesn't > change the permission, but the user doesn't know about it. > {code} > if ((actions == null) || (actions.length == 0)) { > LOG.warn("No actions associated with user > '"+Bytes.toString(userPerm.getUser())+"'"); > return; > } > {code} > I think we should throw an exception instead of just logging. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira