You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2013/01/02 22:20:12 UTC

[jira] [Commented] (ACCUMULO-677) Remove (deprecate) createUser call with authorizations argument

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

Hudson commented on ACCUMULO-677:
---------------------------------

Integrated in Accumulo-Trunk #599 (See [https://builds.apache.org/job/Accumulo-Trunk/599/])
    ACCUMULO-677 - cleaning up shell for these changes (Revision 1427997)
ACCUMULO-677 - deprecated createUser(user,pass,auths). Still uses same code base for thrift, which uses an empty Authorizations (Revision 1427994)

     Result = UNSTABLE
vines : 
Files : 
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/CreateUserCommand.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/ScanCommand.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/SetAuthsCommand.java

vines : 
Files : 
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/SecurityOperations.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/SecurityOperationsImpl.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mock/MockInstance.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mock/MockSecurityOperations.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/CreateUserCommand.java
* /accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mock/MockConnectorTest.java
* /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/PermissionsTest.java
* /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/concurrent/CreateUser.java
* /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/CreateUser.java
* /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java
* /accumulo/trunk/test/src/test/java/org/apache/accumulo/test/MiniAccumuloClusterTest.java

                
> Remove (deprecate) createUser call with authorizations argument
> ---------------------------------------------------------------
>
>                 Key: ACCUMULO-677
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-677
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 1.4.1, 1.4.2
>            Reporter: John Vines
>            Assignee: John Vines
>            Priority: Minor
>              Labels: acl, alter, api, create, permissions, security, user
>             Fix For: 1.5.0
>
>
> Creating a user depends on a different ACL than granting Authorizations. If the user can do one, but not the other it will still create the user but float back an error. This can be confusing to end users, so I think we should isolate createUser to just creating the user. They can then be granted authorizations as need be.

--
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