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