epugh commented on PR #2778: URL: https://github.com/apache/solr/pull/2778#issuecomment-2427166378
Of the two URI builders, I think we would want the jakarta since at some point Apache HTTP Client goes away. What about just looking for `//` and removing one of them? In the 9x line, does that get us through the problem. I remember when we looked at the hostContext a while ago and the reason we changed to just requiring `/solr` was because we thought it wasn't properly supported to use anything else anyway in Solr. Like, it might pass in the tests, but in the real world it would fail. What if we just "fix" the `BaseDistributedSolrTestCase` to not do anything but `/solr`..... -- 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
