New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 893994 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Oct 11
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Downloading filename that contains comma

Reported by sbz...@gmail.com, Oct 10

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36

Example URL:

Steps to reproduce the problem:
1. Download file that contains comma in filename
2. 
3. 

What is the expected behavior?
To download file correctly

What went wrong?
Broswer returned error "ERR_RESPONSE_HEADERS_MULTIPLE_CONTENT_DISPOSITION"

Did this work before? Yes dunno

Chrome version: 69.0.3497.100  Channel: stable
OS Version: 10.0
Flash Version: 

This was a known issue in 2012 and was fixed. https://jira.nuxeo.com/browse/NXP-9542.
see article https://foocompelsyou.wordpress.com/2012/10/01/the-curious-case-of-chrome-content-disposition-and-the-comma/
 
Issues not happening on Firefox or Edge
I assume it's not in a quoted string?  Per spec, commas must be in quoted strings in content-disposition fields.  https://tools.ietf.org/html/rfc6266 (See definition of value.  token is defined in RFC 2616).
Labels: Needs-Feedback
Thanks for reporting this issue.
Please collect and attach a chrome://net-export log. Instructions can be found here: https://sites.google.com/a/chromium.org/dev/for-testers/providing-network-details
Labels: Needs-Triage-M69 Needs-Bisect
The issue was that the comma was not in a quoted string. 
per spec, it is mentioned in Appendix D (page 13) of https://tools.ietf.org/html/rfc626. thank you mmenke@chromium.org!

It was not easy to figure out that chrome considers a comma a disallowed character. Not at all clear and searching online does not show that info. e.g. https://en.wikipedia.org/wiki/Filename 
Where is that info available (besides for the quite obscure reference in https://tools.ietf.org/html/rfc626)?

Project Member

Comment 6 by sheriffbot@chromium.org, Oct 11

Cc: jselover@chromium.org
Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
RFCs (And WHATWG "living standard" docs) are the official standards that all browsers are based on, at least in terms of network protocols.  They specify HTTP, DNS, FTP, and other network protocols.  It's actually in the grammar rule (Second 4.1) of https://tools.ietf.org/html/rfc6266.  It's also in the obsolete https://tools.ietf.org/html/rfc2616 (Section 19.5.1), which is the original HTTP/1.1 specification.

Commas unfortunately have special meaning in HTTP headers, and, with a few exceptions, are generally only unquoted when "Foo: a, b" is the same as:

Foo: a
Foo: b

Since, per spec, it's safe for proxies and browsers to merge the latter into the former.
Cc: annevank...@gmail.com mmenke@chromium.org
Also worth noting that searching for: content-disposition spec

Will get you a link to the RFC as well as an older version of the spec (Which is, unfortunately, is even older than RFC 2616 and does not include an explicit rule about what values should look like).

The reason you get a somewhat confusing "ERR_RESPONSE_HEADERS_MULTIPLE_CONTENT_DISPOSITION" is the aforementioned comma line-merging behavior.

Anyhow, I'm going to go ahead and WontFix this for the moment.  I think that Chrome's behavior is the more correct one here, and reluctant to be more lax in what we accept here.

[+annevankesteren]:  FYI.  Feel free to weigh in (Or not).
Status: WontFix (was: Unconfirmed)
(I agree with your analysis.)

Sign in to add a comment