Skip to content

Commit

Permalink
Fixing ZNodeGroupAclProvider matches implementation for NPE for conne…
Browse files Browse the repository at this point in the history
…ction object (#69)

Co-authored-by: Rahul Rane <[email protected]>
  • Loading branch information
rahulrane50 and rahulrane50 authored May 3, 2022
1 parent 405b055 commit 97a770a
Showing 1 changed file with 3 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -118,14 +118,9 @@ public KeeperException.Code handleAuthentication(ServerObjs serverObjs, byte[] a

@Override
public boolean matches(ServerObjs serverObjs, MatchValues matchValues) {
for (Id id : serverObjs.getCnxn().getAuthInfo()) {
// Not checking for super user here because the check is already covered
// in checkAcl() in ZookeeperServer.class
if (id.getId().equals(matchValues.getAclExpr())) {
return true;
}
}
return false;
// Not checking for super user here because the check is already covered
// in checkAcl() in ZookeeperServer.class
return matchValues.getId().equals(matchValues.getAclExpr());
}

@Override
Expand Down

0 comments on commit 97a770a

Please sign in to comment.