Skip to content

Conversation

@annevk
Copy link
Member

@annevk annevk commented Jan 14, 2026

While it is less likely you can attack a 416, it's probably good to stay consistent.

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

While it is less likely you can attack a 416, it's probably good to stay consistent.
@yoshisatoyanagisawa
Copy link

lgtm.

annevk added a commit to web-platform-tests/wpt that referenced this pull request Jan 17, 2026
See whatwg/fetch#1907 for context.

We use an image instead of JavaScript as JavaScript requires a 2xx status whereas images don't care.

Also remove a redundant null check from useStoredRangeResponse() and a redundant catch() elsewhere.
annevk added a commit to web-platform-tests/wpt that referenced this pull request Jan 20, 2026
See whatwg/fetch#1907 for context.

We use an image instead of JavaScript as JavaScript requires a 2xx status whereas images don't care.

Also remove a redundant null check from useStoredRangeResponse() and a redundant catch() elsewhere.
@annevk annevk merged commit d9fc82f into main Jan 20, 2026
2 checks passed
@annevk annevk deleted the block-416-when-we-block-206 branch January 20, 2026 08:12
KurtCattiSchmidt pushed a commit to KurtCattiSchmidt/wpt-ahem that referenced this pull request Jan 21, 2026
See whatwg/fetch#1907 for context.

We use an image instead of JavaScript as JavaScript requires a 2xx status whereas images don't care.

Also remove a redundant null check from useStoredRangeResponse() and a redundant catch() elsewhere.
lando-prod-mozilla bot pushed a commit to mozilla-firefox/firefox that referenced this pull request Jan 23, 2026
…range requests, a=testonly

Automatic update from web-platform-tests
Fetch: blocking opaque 416 responses to range requests

See whatwg/fetch#1907 for context.

We use an image instead of JavaScript as JavaScript requires a 2xx status whereas images don't care.

Also remove a redundant null check from useStoredRangeResponse() and a redundant catch() elsewhere.

--

wpt-commits: e3661ee657a61122b3026e8eaaba31cbddd7a9a8
wpt-pr: 57225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants