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

Missing WebSockets (/ws, /wss) Transport Specification #594

Closed
lidel opened this issue Nov 10, 2023 · 2 comments · Fixed by #624
Closed

Missing WebSockets (/ws, /wss) Transport Specification #594

lidel opened this issue Nov 10, 2023 · 2 comments · Fixed by #624

Comments

@lidel
Copy link
Member

lidel commented Nov 10, 2023

This is a placeholder issue, as we seem to be missing specification for /ws and /wss.

Which codebase/package would be the best "source of truth" to start documenting things?
@mxinden @marten-seemann @sukunrt

@mxinden
Copy link
Member

mxinden commented Nov 10, 2023

Which codebase/package would be the best "source of truth" to start documenting things?

No opinion. You can find the Rust implementation here. Note however the missing support for URL paths.

@MarcoPolo
Copy link
Contributor

Which codebase/package would be the best "source of truth" to start documenting things?

Probably js <-> go. But they should all be interoperable according to our tests. The path to connect to is not well defined and may benefit from the httppath proposal.

@dhuseby dhuseby moved this to Triage in libp2p Specs May 7, 2024
achingbrain added a commit that referenced this issue Aug 13, 2024
Specifies the WebSockets transport with notes on certificates,
multiplexing, etc.

Fixes #594
@github-project-automation github-project-automation bot moved this from Todo to Done in @lidel's IPFS wishlist Oct 23, 2024
@github-project-automation github-project-automation bot moved this from Triage to Done in libp2p Specs Oct 23, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in @lidel's IPFS wishlist Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants