-
Notifications
You must be signed in to change notification settings - Fork 16
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
Remove references to unknown / missing "Braid-WebRTC" from http spec #95
Comments
Yeah, we've implemented a braid webrtc transport in redwood, but the point of this paragraph isn't to get people using a common webrtc specification, but just to suggest that they could use additional network channels like webrtc as an example. Would it be better if the text said this:
|
Thats a bit better, but I think it still doesn't address my main concern. My concern is that the spec doesn't specify how to embed braid in WebRTC, or how to use braid in a P2P context. This example implies that it has. The question to ask about a good spec is, "with no supplementary material, could I read the spec and know how to write an implementation that can talk to any other implementation". The answer is "no" wrt WebRTC and P2P. (And if we want to describe how to do that, that description shouldn't be described as an example. This is an island we want to reach, but the spec doesn't build a bridge to that island yet. We should either call that out explicitly ("Future versions of this protocol may...") or remove this text completely. |
Spec says:
Its really jarring mentioning a separate, unspecified protocol (Braid-WebRTC) in this example. This reference is confusing as-is, because people will go looking for that spec!
Either the text should be clear about whats being proposed here. Eg:
Or remove / change this example entirely.
The text was updated successfully, but these errors were encountered: