Closed Bug 656251 Opened 9 years ago Closed 8 years ago

reduce verbosity of Accept header for fetching XSLT

Categories

(Core :: XSLT, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla9

People

(Reporter: julian.reschke, Assigned: julian.reschke)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

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
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86 → All
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
Attachment #551422 - Flags: review?
Attachment #551422 - Flags: review? → review?(jonas)
Keywords: checkin-needed
Assignee: nobody → julian.reschke
http://hg.mozilla.org/mozilla-central/rev/f1820278311e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
I don't believe this requires any developer-visible documentation.
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.
You need to log in before you can comment on or make changes to this bug.