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

Websocket Transport #34

Open
bradrydzewski opened this issue Feb 2, 2016 · 3 comments
Open

Websocket Transport #34

bradrydzewski opened this issue Feb 2, 2016 · 3 comments

Comments

@bradrydzewski
Copy link

Not sure this is in-scope for what you're doing, but looks like there is draft support for websockets as a protocol which seems really interesting http://www.ietf.org/id/draft-savolainen-core-coap-websockets-05.txt

@dubek
Copy link
Collaborator

dubek commented Feb 5, 2016

If a device is powerful enough (CPU/memory) to handle TCP+HTTP (for websockets), why would it need CoAP? I thought that CoAP was intended for those constrained devices that don't want to handle TCP state and HTTP parsing.

@kulak
Copy link

kulak commented Feb 11, 2016

Some devices have battery life restrictions, so web sockets will not work well.

@bradrydzewski
Copy link
Author

I think the draft rfc describes some interesting use cases for websockets http://www.ietf.org/id/draft-savolainen-core-coap-websockets-05.txt but implementing such a feature in this library would largely depend on that draft being accepted and the author of this library being interested.

Some devices have battery life restrictions, so web sockets will not work well.

I'm not suggesting this be the default implementation. I do think go provides some nice interfaces with io.Reader and io.Writer that might allow the underlying transport to be separated from the implementation. This is something you see in the underlying Go rpc package which support rpc.NewClient(conn io.ReadWriteCloser) *Client

But again, just a suggestion and the owner can feel free to close if this is out of scope for the library, or if the draft spec for websockets has been rejected.

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

No branches or pull requests

3 participants