Address some comment 2

Signed-off-by: Ryan Liang <jiallian@amazon.com>
peternied · Aug 22, 2023 · b2c7d75 · b2c7d75
1 parent d79973c
commit b2c7d75
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java b/src/main/java/org/opensearch/security/action/onbehalf/CreateOnBehalfOfTokenAction.java
@@ -110,7 +110,7 @@ public void accept(RestChannel channel) throws Exception {
                 try {
                     if (vendor == null) {
                         channel.sendResponse(
-                            new BytesRestResponse(RestStatus.SERVICE_UNAVAILABLE, "on_behalf_of configuration is not being configured")
+                            new BytesRestResponse(RestStatus.SERVICE_UNAVAILABLE, "on_behalf_of is either disabled or the configuration is invalid")
                         );
                         return;
                     }

diff --git a/src/main/java/org/opensearch/security/http/OnBehalfOfAuthenticator.java b/src/main/java/org/opensearch/security/http/OnBehalfOfAuthenticator.java
@@ -211,8 +211,8 @@ private AuthCredentials extractCredentials0(final RestRequest request) {
             }
 
             final String issuer = claims.getIssuer();
-            final String clusterID = OpenSearchSecurityPlugin.getClusterName().getClusterName().value();
-            if (!issuer.equals(clusterID)) {
+            final String clusterName = OpenSearchSecurityPlugin.getClusterName().getClusterName().value();
+            if (!issuer.equals(clusterName)) {
                 log.error("This issuer of this OBO does not match the current cluster identifier");
                 return null;
             }