Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ping/Pong in WebSockets are not plain text #54

Open
nicola opened this issue Sep 28, 2015 · 4 comments
Open

Ping/Pong in WebSockets are not plain text #54

nicola opened this issue Sep 28, 2015 · 4 comments
Labels

Comments

@nicola
Copy link
Member

nicola commented Sep 28, 2015

The ping in the websocket is a special character 0x9/0x10, while in the implementation I saw "ping"/"pong"

case "ping":

Maybe that is already implemented in "golang.org/x/net/websocket" or maybe this is how it is implemented, ping me back

@nicola nicola changed the title Ping in web sockets is never plain text Ping/Pong in WebSockets are not plain text Sep 28, 2015
@presbrey
Copy link
Member

This functionality is a application-level NOOP I created during browser JS integration/development.

@nicola
Copy link
Member Author

nicola commented Sep 28, 2015

Perfect!
Do you think we should take it out? Should we implement ping/pong as 0x9/0x10?

(ps: nice to e-meet you @presbrey)

@presbrey
Copy link
Member

If we are looking to polish/clean things up, there are much lower hanging fruit imho :)

The x/websocket library handles the 0x9/0x10 at a lower protocol level that is hard to see when you are developing in the browser, just as TCP keepalives exist further below that.

Why would remove it? Is there a new standard for linked data updates over websockets? Does it conflict with a new protocol you want to try?

@nicola
Copy link
Member Author

nicola commented Sep 28, 2015

Oh no no, I just saw it as something "extra" and not needed (and I thought we were trying to achieve what x/websocket does - and I didnt know it did)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants