[ https://issues.apache.org/jira/browse/ZOOKEEPER-1782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16049222#comment-16049222 ]
ASF GitHub Bot commented on ZOOKEEPER-1782: ------------------------------------------- GitHub user revans2 opened a pull request: https://github.com/apache/zookeeper/pull/282 ZOOKEEPER-1782: Let a SASL super user be super You can merge this pull request into a Git repository by running: $ git pull https://github.com/revans2/zookeeper ZOOKEEPER-1782 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/zookeeper/pull/282.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #282 ---- commit 62ee0bcedbbbf34b6baf6effac24765d714928cc Author: Robert Evans <ev...@yahoo-inc.com> Date: 2017-06-14T14:13:03Z ZOOKEEPER-1782: Let a SASL super user be super ---- > zookeeper.superUser is not as super as superDigest > -------------------------------------------------- > > Key: ZOOKEEPER-1782 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1782 > Project: ZooKeeper > Issue Type: Bug > Affects Versions: 3.4.5 > Reporter: Robert Joseph Evans > Assignee: Robert Joseph Evans > Attachments: zk-1782.patch, zk-1782.patch > > > The zookeeper.superUser system property does not fully grant super user > privileges, like zookeeper.DigestAuthenticationProvider.superDigest does. > zookeeper.superUser only has as many privileges as the sasl ACLs on the znode > being accessed. This means that if a znode only has digest ACLs > zookeeper.superUser is ignored. Or if a znode has a single sasl ACL that > only has read privileges zookeeper.superUser only has read privileges. > The reason for this is that SASLAuthenticationProvider implements the > superUser check in the matches method, instead of having the super user > include a new Id("super","") as Digest does. -- This message was sent by Atlassian JIRA (v6.4.14#64029)