Bugfix: Stargate service node registration path must be in lowercase (b…

…aidu#96) Currently only the service name is converted to lowercase. We should do the same on group and version as well.
wangsan · Jul 5, 2019 · c85b666 · c85b666
1 parent a4e8094
commit c85b666
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...okeeper/src/main/java/com/baidu/brpc/naming/zookeeper/StargateZookeeperNamingService.java b/...okeeper/src/main/java/com/baidu/brpc/naming/zookeeper/StargateZookeeperNamingService.java
@@ -193,10 +193,10 @@ public void unregister(RegisterInfo info) {
     }
 
     /**
-     * stargate 注册或者订阅节点时，serviceName 为全小写
+     * stargate 注册或者订阅节点时，整个 Path 为小写
      */
     private String buildParentNodePath(String group, String serviceName, String version) {
-        return "/" + group + ":" + serviceName.toLowerCase() + ":" + version;
+        return ("/" + group + ":" + serviceName + ":" + version).toLowerCase();
     }