may we have to implement a timeout when reconnecting about dubbo-js HOT 5 CLOSED

cool!

from dubbo-js.

welcome pull request 😃

from dubbo-js.

我是#5757的提出者, 我当时的诉求是我的某个应用里connection timeout比较短, 但是socket timeout比较长, 但是目前nodejs并不支持.
你们估计需要在 _initSocket 里面通过global.setTimeout(不是socket.setTimeout)去注册一下onClose, 然后onConnect里clearTimeout.

from dubbo-js.

@DanielYWoo 感谢提醒，不过我刚刚去看了源码，socket.setTimeout 应该大体上和你建议的方式差不多，调用过程如下：

• https://github.com/nodejs/node/blob/1447a79dc435578bffba32ead9b2ffa0425fc30a/lib/net.js#L452
• https://github.com/nodejs/node/blob/4aca277f16b8649b5fc21d41f340fad0a47c2e61/lib/internal/stream_base_commons.js#L240
• https://github.com/nodejs/node/blob/4aca277f16b8649b5fc21d41f340fad0a47c2e61/lib/internal/timers.js#L345

到了这边会发现会实例化一个定时器，而这个定时器在未来 stream.onRead 或者 stream.onWrite 的时候，会被刷新，见：

• https://github.com/nodejs/node/blob/4aca277f16b8649b5fc21d41f340fad0a47c2e61/lib/internal/stream_base_commons.js#L165

• https://github.com/nodejs/node/blob/4aca277f16b8649b5fc21d41f340fad0a47c2e61/lib/internal/stream_base_commons.js#L92

所以我感觉运行结果应该会和 nodejs/node#5757 (comment) 中 bnoordhuis 的结果一样

from dubbo-js.

👍

from dubbo-js.