此处为何要再做一次 decodeURIComponent

[apache]

此处为何要再做一次 decodeURIComponent

[hufeng]

hi @ImHype 感谢你的review 😆稍等我 细细看下

[hufeng]

❯ zkCli -server localhost:2181
Connecting to localhost:2181
Welcome to ZooKeeper!
JLine support is enabled

WATCHER::

WatchedEvent state:SyncConnected type:None path:null
[zk: localhost:2181(CONNECTED) 0] ls /dubbo/com.alibaba.dubbo.demo.DemoProvider/providers
[dubbo%3A%2F%2F192.168.1.6%3A20880%2Fcom.alibaba.dubbo.demo.DemoProvider%3Fanyhost%3Dtrue%26application%3Ddemo-provider%26dubbo%3D2.5.7%26generic%3Dfalse%26interface%3Dcom.alibaba.dubbo.demo.DemoProvider%26methods%3DsayHello%2Ctest%2Cecho%2CgetUserInfo%26pid%3D32593%26revision%3D1.0.0%26side%3Dprovider%26timestamp%3D1526645662790%26version%3D1.0.0, dubbo%3A%2F%2F192.168.1.6%3A20881%2Fcom.alibaba.dubbo.demo.DemoProvider%3Fanyhost%3Dtrue%26application%3Ddemo-provider%26dubbo%3D2.5.7%26generic%3Dfalse%26interface%3Dcom.alibaba.dubbo.demo.DemoProvider%26methods%3DsayHello%2Ctest%2Cecho%2CgetUserInfo%26pid%3D32593%26revision%3D1.0.0%26side%3Dprovider%26timestamp%3D1526645663463%26version%3D1.0.0]
[zk: localhost:2181(CONNECTED) 1]

dubbo在zookeeper中注册的服务是encode过后的。

[ImHype]

https://github.com/dubbo/dubbo2.js/blob/a61a6d1b2c689aba837caa48b63dd6f34017c4e9/packages/dubbo/src/zookeeper.ts#L82

parseUrl 之前都经过了 _getProviderList ，这个函数里面已经做过一次 decodeComponent 了，所以做第二次 decode 没有意义吧，可能还会出现问题

[hufeng]

@ImHype yes,是这样的，没有意义，这个decode最初就是在parseUrl中，后来重构的时候getProviderList做了，这样好判断是不是dubbo协议开头，因为之前我们内部还暴露过jsonrpc://这样的协议，感谢 我来改下代码 ^_^

[ImHype]

@hufeng
哈哈，不如让我蹭个 pr

[hufeng]

@ImHype 来来来 一起玩

[hufeng]

@ImHype 没有等来 先pr了 😆

[ImHype]

repo clone 太久了，在家里忘没 clone 下来

[ImHype]

另： 才发现是我把 dubbo.js 的坑给占了

[hufeng]

@ImHype 估计是因为我放了pdf的分享文件在ppt目录这个要上m了 所以仓库比较大一点。ps. 我们有相同的理想 😆 多指点指点 我们写的还不够好。

[ImHype]

@hufeng 要不要把坑留出来给你们，毕竟你的已经在 dubbo 的仓库下面了，而且也做了很多东西了

[hufeng]

@ImHype 3ks，暂时不要了吧，也方便你后面发npm包，这个东西就是一个名字，没有什么含义，我们专心把项目做好就ok了。

[ImHype]

👏👏👏