Skip to content

Commit

Permalink
HBASE-25543 When configuration hadoop.security.authorization is set t…
Browse files Browse the repository at this point in the history
…o false, the system will still try to authorize an RPC and raise AccessDeniedException (#2932) (#2919)

Signed-off-by: Viraj Jasani <vjasani@apache.org>
Signed-off-by: Reid Chan <reidchan@apache.org>
  • Loading branch information
YutSean committed Feb 7, 2021
1 parent f52b457 commit 7d80d99
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -1973,7 +1973,7 @@ private void processOneRpc(ByteBuffer buf) throws IOException, InterruptedExcept
} else {
processConnectionHeader(buf);
this.connectionHeaderRead = true;
if (!authorizeConnection()) {
if (authorize && !authorizeConnection()) {
// Throw FatalConnectionException wrapping ACE so client does right thing and closes
// down the connection instead of trying to read non-existent retun.
throw new AccessDeniedException("Connection from " + this + " for service " +
Expand Down

0 comments on commit 7d80d99

Please sign in to comment.