1455614 - Comma-separated Content-Length with unequal values should be rejected

Status: RESOLVED FIXED

(Core :: Networking: HTTP, enhancement, P2)

Description

[Anne (:annevk)]

See tests at https://github.com/w3c/web-platform-tests/pull/10548.

I think at the very least we should start rejecting values such as

  Content-Length: 42,30

while continuing to support

  Content-Length: 42,42

As long as we don't, we would parse the above differently from

  Content-Length: 42
  Content-Length: 30

which seems suboptimal.

Comment 1

[Valentin Gosu [:valentin] (he/him)]

The Content-Length parsing is here

WPT here

Comment 2

[Sunil Mayya]

Attachment: Bug 1455614 - improve content-length header parsing. r=#necko

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9865f9ad00a4
improve content-length header parsing. r=necko-reviewers,valentin

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/40282 for changes under testing/web-platform/tests

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/9865f9ad00a4

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot