[Backport 2.x] [Chore] Remove unnecessary return statements (opensear…

…ch-project#4572) Signed-off-by: 10000-ki <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Craig Perkins <[email protected]>
stephen-crawford · Jul 17, 2024 · e8861cf · e8861cf
1 parent 3dd5d9f
commit e8861cf
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/src/main/java/org/opensearch/security/filter/SecurityRestFilter.java b/src/main/java/org/opensearch/security/filter/SecurityRestFilter.java
@@ -254,7 +254,6 @@ void authorizeRequest(RestHandler original, SecurityRequestChannel request, User
                 log.debug(err);
 
                 request.queueForSending(new SecurityResponse(HttpStatus.SC_UNAUTHORIZED, err));
-                return;
             }
         }
     }
@@ -309,7 +308,6 @@ public void checkAndAuthenticateRequest(SecurityRequestChannel requestChannel) t
             if (!registry.authenticate(requestChannel)) {
                 // another roundtrip
                 org.apache.logging.log4j.ThreadContext.remove("user");
-                return;
             } else {
                 // make it possible to filter logs by username
                 org.apache.logging.log4j.ThreadContext.put(