SOLR-16150 Provide correct bound zookeeper interface (apache#802)

Instead of always giving localhost, we can return the interface that ZK actually listens on. Co-authored-by: Kevin Risden <[email protected]>
cowpaths · Oct 18, 2022 · c80cf5b · c80cf5b
1 parent 32e0f68
commit c80cf5b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
@@ -82,7 +82,8 @@ Build
 
 Other Changes
 ---------------------
-(No changes)
+* SOLR-16150: Embedded ZK Server used for zkRun should advertise itself on specific bound interfaces instead of
+  generically on localhost. (Mike Drob)
 
 ==================  9.1.0 ==================
 

diff --git a/solr/core/src/java/org/apache/solr/cloud/SolrZkServer.java b/solr/core/src/java/org/apache/solr/cloud/SolrZkServer.java
@@ -70,7 +70,9 @@ public String getClientString() {
 
     // if the string wasn't passed as zkHost, then use the standalone server we started
     if (zkRun == null) return null;
-    return "localhost:" + zkProps.getClientPortAddress().getPort();
+
+    InetSocketAddress addr = zkProps.getClientPortAddress();
+    return addr.getHostString() + ":" + addr.getPort();
   }
 
   public void parseConfig() {