senthh commented on code in PR #1669:
URL: https://github.com/apache/solr/pull/1669#discussion_r1209726203
##########
solr/core/src/java/org/apache/solr/core/CoreContainer.java:
##########
@@ -547,7 +552,12 @@ private synchronized void initializeAuthenticationPlugin(
// Initialize the plugin
if (pluginClassName != null) {
- log.info("Initializing authentication plugin: {}", pluginClassName);
+ if (authPluginList.contains(pluginClassName)) {
+ log.info("Initializing authentication plugin: {}", pluginClassName);
+ } else {
+ log.warn("Wrong Authentication plugin {} configured." +
Review Comment:
I am aiming this PR just to validate solr's inbuilt Auth plugins and write
WARN logs if it is not matching as per Solr 9+ documentation. It will help
users to, easily, find what mistake they commited.
So I think changing the WARN messages like "Authentication plugin {}
configured is not matching with in-built plugins."
--
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: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
