epugh commented on code in PR #1808:
URL: https://github.com/apache/solr/pull/1808#discussion_r1276584581
##########
solr/core/src/java/org/apache/solr/cli/SolrCLI.java:
##########
@@ -383,6 +383,11 @@ public static boolean exceptionIsAuthRelated(Exception
exc) {
}
public static SolrClient getSolrClient(String solrUrl) {
+ // today we require all urls to end in /solr, however in the future we
will need to support the
+ // /api url end point instead.
+ if (!solrUrl.endsWith(("/solr"))) {
Review Comment:
Ugh... One test has "http://127.0.0.1:51902/uf_"; as the URL.....
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
