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
