Skip to content

Updating explainer#716

Merged
jan-ivar merged 5 commits intomainfrom
wilaw-patch-10
Jan 21, 2026
Merged

Updating explainer#716
jan-ivar merged 5 commits intomainfrom
wilaw-patch-10

Conversation

@wilaw
Copy link
Contributor

@wilaw wilaw commented Jan 7, 2026

The explainer was last updated 3 years ago. This PR updates it to include new features and capabilities added since that time.

Fixes #694

wilaw added 3 commits January 7, 2026 18:28
Revise the WebTransport explainer to add in all the new features and capabilities since the last update in Oct 2021
Expanded the proposed solution section to include subprotocol negotiation details and added a note on additional use cases and security considerations.
Updated the explainer document to clarify WebTransport features and use cases, including new sections on connection management, stream handling, and datagram management.
@wilaw wilaw requested a review from jan-ivar January 7, 2026 17:11
@wilaw wilaw self-assigned this Jan 7, 2026
Copy link
Member

@jan-ivar jan-ivar left a comment

Choose a reason for hiding this comment

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

Overall much improved, thanks! Some nits, feedback on examples. But I'd like to keep the existing problem statement, if possible.

Comment on lines +72 to +74
value: new Uint8Array([0xed, 0xb0, 0x3e, ...]) // For local dev
}],
headers: { 'Authorization': 'Bearer <token>' }
Copy link
Member

Choose a reason for hiding this comment

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

Great examples! But I'd prefer they be slightly more complete, as we have some feedback that people struggle to understand this part otherwise

Copy link
Member

Choose a reason for hiding this comment

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

I can do this in a follow-up

Comment on lines 84 to 92
// Receiving a server-initiated unidirectional stream
const reader = transport.incomingUnidirectionalStreams.getReader();
const { value: stream } = await reader.read();
const data = await new Response(stream).arrayBuffer();

// Creating a bidirectional stream
const biStream = await transport.createBidirectionalStream();
const biWriter = biStream.writable.getWriter();
await biWriter.write(new TextEncoder().encode("Hello Server"));
Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer we use the streams spec as intended with pipeTo like example 7 instead of getReader and getWriter. I can also do this in a follow-up.

Applies throughout

wilaw and others added 2 commits January 21, 2026 17:26
Updated references from 'transport' to 'wt' for consistency and clarity. Enhanced explanations regarding WebTransport's capabilities and use cases.
Co-authored-by: Jan-Ivar Bruaroey <jan-ivar@users.noreply.github.com>
@wilaw wilaw added Editors can integrate Editorial This is an editorial PR that can be merged by editors without discussion labels Jan 21, 2026
@jan-ivar jan-ivar merged commit 62374ed into main Jan 21, 2026
2 checks passed
github-actions bot added a commit that referenced this pull request Jan 21, 2026
SHA: 62374ed
Reason: push, by jan-ivar

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Editorial This is an editorial PR that can be merged by editors without discussion Editors can integrate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update explainer to cover recent features

2 participants