Fix variable name

Signed-off-by: Ryan Liang <jiallian@amazon.com>
opensearch-project · May 10, 2023 · b88123b · b88123b
1 parent 277a999
commit b88123b
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/main/java/org/opensearch/security/http/HTTPOnBehalfOfJwtAuthenticator.java b/src/main/java/org/opensearch/security/http/HTTPOnBehalfOfJwtAuthenticator.java
@@ -190,12 +190,12 @@ private AuthCredentials extractCredentials0(final RestRequest request) {
                 final String rolesClaim = rolesObject.toString();
 
                 // Extracting roles based on the compatbility mode
-                String rolesPostEncryptionDecryption = rolesClaim;
+                String decryptedRoles = rolesClaim;
                 if (rolesObject == claims.get("er")) {
                     //TODO: WHERE TO GET THE ENCRYTION KEY
-                    rolesPostEncryptionDecryption = EncryptionDecryptionUtil.decrypt(encryptionKey, rolesClaim);
+                    decryptedRoles = EncryptionDecryptionUtil.decrypt(encryptionKey, rolesClaim);
                 }
-                roles = Arrays.stream(rolesPostEncryptionDecryption.split(",")).map(String::trim).toArray(String[]::new);
+                roles = Arrays.stream(decryptedRoles.split(",")).map(String::trim).toArray(String[]::new);
             }
 
             if (subject == null) {
@@ -228,9 +228,6 @@ private AuthCredentials extractCredentials0(final RestRequest request) {
 
     @Override
     public boolean reRequestAuthentication(final RestChannel channel, AuthCredentials creds) {
-        final BytesRestResponse wwwAuthenticateResponse = new BytesRestResponse(RestStatus.UNAUTHORIZED,"");
-        wwwAuthenticateResponse.addHeader("WWW-Authenticate", "Bearer realm=\"OpenSearch Security\"");
-        channel.sendResponse(wwwAuthenticateResponse);
         return false;
     }