realglobe-Inc / v-call

CLI caller for v
MIT License
1 stars 0 forks source link

ときどきブロックする #1

Open miminashi opened 7 years ago

miminashi commented 7 years ago
$ v-call -H "v.realglobe.work" -P "https" "jp.realglobe.daisan.humidifier-02" uname
hoge
^D
hoge hoge
miminashi commented 7 years ago

いま再度試してみたらブロックしませんでした。 objectの有無は関係ないみたいです。

miminashi@mmns2016:v-call$ node bin/v-call -H "v.realglobe.work" -P "https" "jp.realglobe.daisan.humidifier-02" uname
Darwin mmns2016.local 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64
okunishinishi commented 7 years ago

@miminashi

wsのエラーハンドリングが抜けていたので修正しました。多分これで直ります。

npm update v-call -gv-call@1.2.3にしてもらえば、固まらずにエラーを吐くようになるはず。

https://github.com/realglobe-Inc/v-spot-ws/commit/ef8878348c44b2685ed8948cd43a4d052d174a70#diff-1dd29de88aac32f048790a7d711ea337R35

okunishinishi commented 7 years ago

以下の対応が必要と思われ

https://github.com/websockets/ws#how-to-detect-and-close-broken-connections

okunishinishi commented 7 years ago

@miminashi 上記の対応を入れました。cloud側から定期的にクライアントにpingを打って失敗したら接続を切ります。これにより、dameon側の接続が切れた後はエラーが吐かれるようになる。はず。