Skip to content

fix(stream): v2.22.2 inject a Range for clients that send none - #89

Merged
picklem0b merged 1 commit into
mainfrom
dev
Sep 28, 2026
Merged

picklem0b merged 1 commit into
mainfrom
dev

Conversation

@picklem0b

Copy link
Copy Markdown
Owner

The experiment branch's live CDN run measured an edge behaviour this stack shares: a range-less request stalls until the client gives up, while bytes=0- on the same URL returns the whole file in one 206. open_upstream now sends bytes=0- on the client's behalf, and the relay presents that whole-file 206 as a plain 200 (a capped 206 keeps the CDN's framing). Pinned by MockTransport tests at the service boundary and presentation tests at the router.

Also: openapi.json had drifted to 2.22.0 while VERSION said 2.22.1 — regenerated so it can never silently lag again; package-lock now carries the web version it should have had since 2.22.1.

The experiment branch's live CDN run measured an edge behaviour this
stack shares: a range-less request stalls until the client gives up,
while bytes=0- on the same URL returns the whole file in one 206.
open_upstream now sends bytes=0- on the client's behalf, and the relay
presents that whole-file 206 as a plain 200 (a capped 206 keeps the
CDN's framing). Pinned by MockTransport tests at the service boundary
and presentation tests at the router.

Also: openapi.json had drifted to 2.22.0 while VERSION said 2.22.1 —
regenerated so it can never silently lag again; package-lock now
carries the web version it should have had since 2.22.1.
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 6faca747-9544-449a-af92-67902500c950

📥 Commits

Reviewing files that changed from the base of the PR and between 9527290 and 33a8c09.

⛔ Files ignored due to path filters (2)
  • api/uv.lock is excluded by !**/*.lock
  • web/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (9)
  • api/app/main.py
  • api/app/routers/stream_router.py
  • api/app/services/stream_service.py
  • api/pyproject.toml
  • api/tests/test_relay_cache_gating.py
  • api/tests/test_stream_service.py
  • web/package.json
  • web/src/lib/constants.ts
  • web/src/types/openapi.json
 _____________________________________________________________________
< Not saying it's bad... but the compiler just asked for a union rep. >
 ---------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@picklem0b
picklem0b merged commit 09fc4e6 into main Sep 28, 2026
9 of 11 checks passed
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.

1 participant