deserializeSafeFromHeader uses context.getHeader(headerName) inst…

…ead of `context.getHeaders()` (#2768) (#2771) (cherry picked from commit 7c4e06d) Co-authored-by: Paras Jain <[email protected]>
opensearch-project · May 15, 2023 · dae8f76 · dae8f76
1 parent 323a3df
commit dae8f76
Showing 1 changed file with 1 addition and 9 deletions.
diff --git a/src/main/java/org/opensearch/security/support/HeaderHelper.java b/src/main/java/org/opensearch/security/support/HeaderHelper.java
@@ -27,7 +27,6 @@
 package org.opensearch.security.support;
 
 import java.io.Serializable;
-import java.util.Map;
 
 import com.google.common.base.Strings;
 
@@ -56,15 +55,8 @@ public static String getSafeFromHeader(final ThreadContext context, final String
             return null;
         }
 
-        String headerValue = null;
-
-        Map<String, String> headers = context.getHeaders();
-        if (!headers.containsKey(headerName) || (headerValue = headers.get(headerName)) == null) {
-            return null;
-        }
-
         if (isInterClusterRequest(context) || isTrustedClusterRequest(context) || isDirectRequest(context)) {
-            return headerValue;
+            return context.getHeader(headerName);
         }
 
         return null;