[jira] Commented: (ZOOKEEPER-49) SetACL does not work

["Benjamin Reed (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606656#action_12606656 ] 

Benjamin Reed commented on ZOOKEEPER-49:
----------------------------------------

Shane was correct. The fixupACL needs to be done after the setAclRequest is deserialized. It's a trivial change. I've included the fix and a test case for it in ZOOKEEPER-48.

> SetACL does not work
> --------------------
>
>                 Key: ZOOKEEPER-49
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-49
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: server
>            Reporter: Benjamin Reed
>
> As reported by Shane:
> Still exploring the ACL stuff in Zookeeper.  Tried using setACL for a  
> path but get InvalidACL error thrown .... looking at pRequest in  
> PrepRequestProcessor ... and in particular these lines ...
>                  SetACLRequest setAclRequest = new SetACLRequest();
>                  if (!fixupACL(request.authInfo,  
> setAclRequest.getAcl())) {
>                      throw new KeeperException(Code.InvalidACL);
>                  }
> a new SetACLRequest will return a null when called in fixupACL  
> returning false and throwing the exception .... as far as I can see.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.