techdocsmith commented on a change in pull request #10532:
URL: https://github.com/apache/druid/pull/10532#discussion_r513714301
##########
File path: docs/development/extensions-core/druid-basic-security.md
##########
@@ -62,16 +69,16 @@
druid.auth.authenticator.MyBasicMetadataAuthenticator.authorizerName=MyBasicMeta
```
To use the Basic authenticator, add an authenticator with type `basic` to the
authenticatorChain.
-The authenticator needs to also define a credentialsValidator with type
'metadata' or 'ldap'.
-If credentialsValidator is not specified, type 'metadata' will be used as
default.
+The authenticator must define a credentials validator `credentialsValidator`
with type 'metadata' or 'ldap'.
+Otherwise the credentials validator defaults to `metadata` if you do not
specify another type.
Review comment:
@nishantmonu51 , @suneet-s updated this to reflect that `metadata` is
the default. And you only need to define one for `LDAP`. PTAL. Not sure if I've
100% got the nuance here.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
