Closed
Bug 656251
Opened 14 years ago
Closed 13 years ago
reduce verbosity of Accept header for fetching XSLT
Categories
(Core :: XSLT, defect)
Core
XSLT
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: julian.reschke, Assigned: julian.reschke)
Details
(Keywords: dev-doc-complete)
Attachments
(1 file)
927 bytes,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:2.0.1) Gecko/20100101 Firefox/4.0.1
Build Identifier:
FF4 sends the following Accept header when fetching an XSLT referenced from an XML resource:
text/xml,application/xml,application/xhtml+xml,*/*;q=0.1
Observations:
- doesn't list text/xsl (the "unofficial" media type)
- doesn't list application/xslt+xml (the official type, see https://bugzilla.mozilla.org/show_bug.cgi?id=655238)
Also, it has a fallback to */* anyway.
Proposal: cut this down to just "*/*".
Comparison with other UAs (see http://lists.w3.org/Archives/Public/www-archive/2011May/0013.html):
IE9: */*
Safari 5: text/xml, application/xml, application/xhtml+xml, text/xsl,
application/rss+xml, application/atom+xml
Opera 11.10: text/html, application/xml;q=0.9, application/xhtml+xml,
image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1
Chrome 12: text/xml, application/xml, application/xhtml+xml, text/xsl,
application/rss+xml, application/atom+xml
(so most of the other UAs are equally weird, except IE9)
(Also note that in theory Accept isn't needed here at all, but there may be servers that rely on its presence)
Reproducible: Always
Assignee | ||
Updated•14 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Updated•13 years ago
|
OS: Windows 7 → All
Hardware: x86 → All
Assignee | ||
Comment 1•13 years ago
|
||
Updated comparison (see <http://lists.w3.org/Archives/Public/www-archive/2011Aug/0007.html>):
Firefox 5:
text/css -> Accept: text/css,*/*;q=0.1
text/xsl -> text/xml,application/xml,application/xhtml+xml,*/*;q=0.1
application/xslt+xml -> not supported
Firefox 7 ("aurora"):
text/css -> text/css,*/*;q=0.1
text/xsl ->
text/xml,application/xml,application/xhtml+xml,application/xslt+xml,*/*;q=0.1
application/xslt+xml ->
text/xml,application/xml,application/xhtml+xml,application/xslt+xml,*/*;q=0.1
IE9:
text/css -> text/css
text/xsl -> */*
application/xslt+xml -> */*
Safari 5.1:
text/css -> text/css,*/*;q=0.1
text/xsl -> text/xml, application/xml, application/xhtml+xml, text/xsl,
application/rss+xml, application/atom+xml
application/xslt+xml -> not supported
Opera 11.50:
text/css -> text/html, application/xml;q=0.9, application/xhtml+xml,
image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1
text/xsl -> text/html, application/xml;q=0.9, application/xhtml+xml,
image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1
application/xslt+xml -> not supported
Chrome 13:
text/css -> text/css,*/*;q=0.1
text/xsl -> text/xml, application/xml, application/xhtml+xml, text/xsl,
application/rss+xml, application/atom+xml
application/xslt+xml -> not supported
Konqueror 4.6:
text/css -> text/css,*/*;q=0.1
text/xsl -> not supported
application/xslt+xml -> not supported
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #551422 -
Flags: review?
Assignee | ||
Updated•13 years ago
|
Attachment #551422 -
Flags: review? → review?(jonas)
Attachment #551422 -
Flags: review?(jonas) → review+
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Updated•13 years ago
|
Assignee: nobody → julian.reschke
Comment 3•13 years ago
|
||
Keywords: checkin-needed
Whiteboard: [inbound]
Comment 4•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
Updated•13 years ago
|
Keywords: dev-doc-needed
Assignee | ||
Comment 5•13 years ago
|
||
I don't believe this requires any developer-visible documentation.
Comment 6•13 years ago
|
||
Just for completeness's sake, I've listed this change on Firefox 9 for developers. The previous header was not documented, though, so this is the only documentation effect.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•