epugh commented on code in PR #1669: URL: https://github.com/apache/solr/pull/1669#discussion_r1209570539
########## 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 wonder if this needs to be more of a "The plugin X doesn't match any of the default plugins shipped with Solr"... Wouldn't there be a possibility of someone writing their own plugin, so it wouldn't be in the authPluginList? -- 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