[GitHub] [pulsar] gaoran10 commented on a diff in pull request #20030: [fix][broker] Fix the reason label of authentication metrics

[via GitHub]

gaoran10 commented on code in PR #20030:
URL: https://github.com/apache/pulsar/pull/20030#discussion_r1176041341


##########
pulsar-broker-auth-oidc/src/main/java/org/apache/pulsar/broker/authentication/oidc/AuthenticationProviderOpenID.java:
##########
@@ -89,7 +89,7 @@ public class AuthenticationProviderOpenID implements 
AuthenticationProvider {
     private static final String SIMPLE_NAME = 
AuthenticationProviderOpenID.class.getSimpleName();
 
     // Must match the value used by the OAuth2 Client Plugin.
-    private static final String AUTH_METHOD_NAME = "token";
+    private static final String AUTH_METHOD_NAME = "openid";

Review Comment:
   Ok, I'll revert this change.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org