Socket.io-client: Документируйте события ping и pong в http://socket.io/docs

Созданный на 13 нояб. 2016 · 10Комментарии · Источник: socketio/socket.io-client

События ping и pong недокументированы на веб-сайте (socket.io/docs), что может вызвать путаницу, когда кто-то пытается реализовать свою собственную систему пинг-понга. socket.emit('pong') в браузере ничего не отправляет через сокет (проверено с помощью localStorage.debug = '*'; ), возможно, socket.emit('ping') также должен сообщать об ошибке, позволяя разработчику узнать, что это зарезервированное ключевое слово .

С 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)."

Источник

joonas-lahtinen

👍6

Самый полезный комментарий

Разве пинг / понг - это не какое-то основное дерьмо в сети? какого черта они не были бы задокументированы? Есть ли события пинга и понга, которые отправляются обратно и четвертое между сервером и клиентом автоматически (при подключении к сокету)? Если да, как мне посмотреть на эти события и их временные метки, чтобы измерить задержку?

ForgeableSum 9 июн. 2017

👍6

Все 10 Комментарий

Согласовано!
Я знаю, что нужно прочитать великолепное руководство, но мне все же удалось потерять пару часов (и волосы), пытаясь понять, почему, черт возьми, мой код не работает.
Совершенно моя вина, но мне бы очень понравилось, если бы мне бросили какое-нибудь предупреждение с надписью «Эти события зарезервированы, попробуйте что-нибудь еще».

hugohil 24 нояб. 2016

PR, как всегда, приветствуется!

darrachequesne 24 нояб. 2016

Я уже подумал об этом, но мне не удалось найти репозиторий, содержащий исходный код веб-сайта. Может я просто некомпетентен или слишком устал, можешь указать мне на это?

joonas-lahtinen 24 нояб. 2016

Я только начал смотреть на это (см. Здесь ), но для этого нужно что-то вроде параметра verbose . Плюс я почти уверен, что это тоже вызвано этими «внутренними» событиями. И, наконец, console.warn.calledWith() не существует в файле expect.
Еще нужна работа ...

hugohil 24 нояб. 2016

Закрытие из-за бездействия, при необходимости откройте снова.

darrachequesne 10 янв. 2017

Трудно сказать, была ли проблема исправлена или нет, из-за # 1054 (страница в настоящее время ничего не отображает).

joonas-lahtinen 10 янв. 2017

ForgeableSum 9 июн. 2017

👍6

Можем ли мы снова открыть это, я согласен с ForgeableSum. У нас должно быть больше документации.

tcaer 20 июн. 2017

👍3

Ну, это уже задокументировано https://github.com/socketio/socket.io-client/blob/master/docs/API.md#event -ping-1, я часами пытался получить данные из пользовательского события ping отправлено моим клиентом, похоже, вы не можете отменить это событие. 😅