Issue metadata
Sign in to add a comment
|
Downloading filename that contains comma
Reported by
sbz...@gmail.com,
Oct 10
|
||||||||||||||||||||||
Issue descriptionUserAgent: 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/
,
Oct 10
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).
,
Oct 10
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
,
Oct 11
,
Oct 11
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)?
,
Oct 11
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
,
Oct 11
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.
,
Oct 11
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).
,
Oct 11
,
Oct 11
(I agree with your analysis.) |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by sbz...@gmail.com
, Oct 10