[SOLR-13727] V2Requests: HttpSolrClient replaces first instance of "/solr" with "/api" instead of using regex pattern

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 8.2
• Fix Version/s: 8.3, 9.0
• Component/s: clients - java, v2 API
• Labels: easyfix patch
• Assignee: Unassigned
• Reporter: Megan Carey
• Created: 29/Aug/19 18:58
• Updated: 12/May/22 00:26
• Resolved: 05/Sep/19 02:28

Description

When the HttpSolrClient is formatting a V2Request, it needs to change the endpoint from the default "/solr/..." to "/api/...". It does so by simply calling String.replace, which replaces the first instance of "/solr" in the URL with "/api".

 

In the case where the host's address starts with "solr" and the HTTP protocol is appended, this call changes the address for the request. Example:

if baseUrl is "http://solr-host.com/8983/solr", this call will change to "http:/api-host.com:8983/solr"

 

We should use a regex pattern to ensure that we're replacing the correct portion of the URL.

Issue Links

• links to: GitHub Pull Request #850