tuteng commented on a change in pull request #14044:
URL: https://github.com/apache/pulsar/pull/14044#discussion_r807788136
##########
File path:
pulsar-broker-common/src/main/java/org/apache/pulsar/broker/authentication/AuthenticationService.java
##########
@@ -96,6 +97,14 @@ public String authenticateHttpRequest(HttpServletRequest
request) throws Authent
String.format("Unsupported authentication method:
[%s].", authMethodName));
}
try {
+ if (authData == null) {
+ // In the default implementation clas
OneStageAuthenticationStates, the authentication has been
+ // done, in order to avoid secondary authentication here
directly return the role, if the user
+ // custom implementation, should consider adding
authentication in their own implementation class.
+ AuthenticationState authenticationState =
providerToUse.newHttpAuthState(request);
Review comment:
fixed
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
