[GitHub] [solr] janhoy commented on pull request #1669: SOLR-16826: Validate if authentication plugin class name are valid

2023-05-31 Thread via GitHub
janhoy commented on PR #1669: URL: https://github.com/apache/solr/pull/1669#issuecomment-1569702974 Closing this, please open a new PR with Upgrade Notes doc changes... -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

[GitHub] [solr] janhoy commented on pull request #1669: SOLR-16826: Validate if authentication plugin class name are valid

2023-05-30 Thread via GitHub
janhoy commented on PR #1669: URL: https://github.com/apache/solr/pull/1669#issuecomment-1568580536 I'm not a fan of this solution. Yet a place of strings to maintain. What is wrong with the ClassNotFound exception you (probably) already get? Does it not tell you clearly the problem