janhoy commented on code in PR #2885:
URL: https://github.com/apache/solr/pull/2885#discussion_r1861295703
##########
solr/core/src/java/org/apache/solr/cli/ExportTool.java:
##########
@@ -660,8 +660,8 @@ class CoreHandler {
}
boolean exportDocsFromCore() throws IOException, SolrServerException {
-
- try (SolrClient client = CLIUtils.getSolrClient(baseurl, credentials))
{
+ // reference the replica's node URL, not the baseUrl in scope, which
could be anywhere
+ try (SolrClient client = CLIUtils.getSolrClient(replica.getBaseUrl(),
credentials)) {
Review Comment:
It’s a corner case for sure and not worth modifying until there is a user
request. Just wanted to highlight the issue and raise awareness as to whether
we want bin/solr to work through reverse proxy.
--
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]
