Skip to content
This repository has been archived by the owner on May 26, 2022. It is now read-only.

Add install, usage & addressing info to README #49

Merged
merged 2 commits into from
Jul 10, 2019
Merged

Conversation

yusefnapora
Copy link
Contributor

This is essentially the same update as in libp2p/go-tcp-transport#41, but for WebSockets.

I linked to the CoC and contributing gif from libp2p/community#3 and libp2p/community#4 respectively, so we should probably do those first.

Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(proxying comment is an optional future improvement)

security for each connection according to the protocols supported by each party.

The transport upgrader also negotiates a stream multiplexing protocol to allow
many bidirectional streams to coexist on a single WebSocket connection.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would also be nice to add a note about proxying. Many HTTP(s) proxies natively support websockets so they're a way to punch out of corporate networks.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good call. I'll add that in on the next round.

@yusefnapora yusefnapora merged commit ec8d181 into master Jul 10, 2019
@yusefnapora yusefnapora deleted the readme-update branch July 10, 2019 18:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants