Socket.io-client: Documentar eventos 'ping' e 'pong' em http://socket.io/docs

Criado em 13 nov. 2016 · 10Comentários · Fonte: socketio/socket.io-client

ping eventos pong estão documentados no site (socket.io/docs), podendo causar confusão quando alguém tenta implementar seu próprio sistema de pingue-pongue. socket.emit('pong') no navegador não faz com que nada seja enviado pelo soquete (verificado com localStorage.debug = '*'; ), talvez socket.emit('ping') também deva relatar um erro informando ao desenvolvedor que é uma palavra-chave reservada .

De README.md:

- `ping`. Fired when a ping packet is written out to the server.
- `pong`. Fired when a pong is received from the server.
    Parameters:
    - `Number` number of ms elapsed since `ping` packet (i.e.: latency)."

Fonte

joonas-lahtinen

👍6

Comentários muito úteis

O ping / pong não é uma merda básica real em rede? por que diabos eles não seriam documentados? Há eventos de ping e pong enviados de volta e quarto entre o servidor e o cliente automaticamente (na conexão de soquete)? Em caso afirmativo, como vejo esses eventos e seus carimbos de data / hora para medir a latência?

ForgeableSum em 9 jun. 2017

👍6

Todos 10 comentários

Concordou!
Sei que é preciso ler o fabuloso manual, mas ainda assim consegui perder algumas horas (e cabelo) tentando descobrir por que diabos meu código não estava funcionando.
Totalmente minha culpa, mas eu teria gostado muito se eu recebesse algum aviso dizendo "Esses eventos estão reservados, tente outra coisa".

hugohil em 24 nov. 2016

PR é bem-vindo, como sempre!

darrachequesne em 24 nov. 2016

Já considerei isso, mas não consegui encontrar um repositório que contenha o código-fonte do site. Talvez eu seja apenas incompetente ou esteja muito cansado, você pode me indicar isso?

joonas-lahtinen em 24 nov. 2016

Acabei de começar a olhar para isso (veja aqui ), mas isso vem com a necessidade de algo como um parâmetro verbose . Além disso, tenho certeza de que isso também é acionado por esses eventos "internos". E finalmente console.warn.calledWith() não existe no esperado.
Ainda preciso de algum trabalho ...

hugohil em 24 nov. 2016

Fechando por inatividade, reabra se necessário.

darrachequesne em 10 jan. 2017

É difícil saber se o problema foi corrigido ou não, devido ao # 1054 (a página não renderiza nada atualmente).

joonas-lahtinen em 10 jan. 2017

ForgeableSum em 9 jun. 2017

👍6

Podemos reabrir isso, eu concordo com o ForgeableSum. Devíamos ter mais documentação.

tcaer em 20 jun. 2017

👍3

Bem, já está documentado https://github.com/socketio/socket.io-client/blob/master/docs/API.md#event -ping-1, Eu estava lutando por horas tentando receber dados de um evento ping personalizado enviado pelo meu cliente, parece que você não pode substituir este evento. 😅