Skip to content

Conversation

@SimaoMoreira5228
Copy link
Contributor

@SimaoMoreira5228 SimaoMoreira5228 commented Oct 8, 2025

Enables handling of WebTransport sessions, uni- and bidirectional streams, and updates dependencies and Bazel integration for WebTransport crate. should solve CLOUD-38

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

❌ Patch coverage is 15.38462% with 264 lines in your changes missing coverage. Please review.
✅ Project coverage is 8.42%. Comparing base (b04a6b6) to head (737074f).

Files with missing lines Patch % Lines
crates/http/src/backend/h3/webtransport.rs 0.00% 205 Missing ⚠️
crates/http/src/backend/h3.rs 42.10% 44 Missing ⚠️
crates/http/src/server.rs 57.14% 12 Missing ⚠️
crates/http/src/body.rs 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #621       +/-   ##
==========================================
- Coverage   61.66%   8.42%   -53.24%     
==========================================
  Files         385     153      -232     
  Lines       31081   12375    -18706     
==========================================
- Hits        19165    1043    -18122     
+ Misses      11916   11332      -584     
Files with missing lines Coverage Δ
crates/http/src/extensions.rs 50.00% <ø> (ø)
crates/http/src/body.rs 80.70% <0.00%> (-4.49%) ⬇️
crates/http/src/server.rs 88.13% <57.14%> (-9.83%) ⬇️
crates/http/src/backend/h3.rs 64.55% <42.10%> (-22.70%) ⬇️
crates/http/src/backend/h3/webtransport.rs 0.00% <0.00%> (ø)

... and 316 files with indirect coverage changes

Components Coverage Δ
scuffle-aac ∅ <ø> (∅)
scuffle-amf0 0.00% <ø> (-90.32%) ⬇️
scuffle-av1 ∅ <ø> (∅)
scuffle-batching ∅ <ø> (∅)
scuffle-bootstrap 0.00% <ø> (-82.92%) ⬇️
scuffle-bytes-util 0.00% <ø> (-96.71%) ⬇️
scuffle-context 80.60% <ø> (-19.40%) ⬇️
scuffle-expgolomb ∅ <ø> (∅)
scuffle-ffmpeg ∅ <ø> (∅)
scuffle-flv ∅ <ø> (∅)
scuffle-future-ext 100.00% <ø> (+50.00%) ⬆️
nutype-enum ∅ <ø> (∅)
scuffle-h264 ∅ <ø> (∅)
scuffle-http 61.06% <15.38%> (-25.49%) ⬇️
scuffle-metrics 50.87% <ø> (-44.16%) ⬇️
postcompile ∅ <ø> (∅)
scuffle-pprof 100.00% <ø> (ø)
scuffle-rtmp 0.00% <ø> (-90.89%) ⬇️
scuffle-settings ∅ <ø> (∅)
scuffle-signal ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2025

🚀 Preview Deployments

Deployment Status Preview URL
dashboard ⏭️ -
docs ⏭️ -
rustdoc https://docs-scuffle-pw7a2pal6-scufflecloud.vercel.app
cloud emails render ⏭️ -

@SimaoMoreira5228 SimaoMoreira5228 marked this pull request as ready for review October 13, 2025 08:13
@SimaoMoreira5228 SimaoMoreira5228 requested review from a team as code owners October 13, 2025 08:13
@TroyKomodo
Copy link
Member

This looks good, are there no webtransport rust client implementations?

@TroyKomodo
Copy link
Member

Perhaps we could do tests with https://docs.rs/web-transport/latest/web_transport/struct.Client.html

@lennartkloock
Copy link
Member

Perhaps we could do tests with https://docs.rs/web-transport/latest/web_transport/struct.Client.html

Sounds good, in this PR or should I create a ticket for it?

@TroyKomodo
Copy link
Member

Perhaps we could do tests with docs.rs/web-transport/latest/web_transport/struct.Client.html

Sounds good, in this PR or should I create a ticket for it?

This pr perhaps @SimaoMoreira5228 can do it?

SimaoMoreira5228 and others added 18 commits October 15, 2025 16:53
Enables handling of WebTransport sessions, uni- and bidirectional streams,
and updates dependencies and Bazel integration for WebTransport crate.
Introduces WebTransport session management with unified handling for bidirectional streams, unidirectional streams, and datagrams over HTTP/3.

Improves WebTransport demo to showcase datagram echo and logging, and refactors backend logic to simplify session integration.
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

Successfully merging this pull request may close these issues.

4 participants