reduce verbosity of Accept header for fetching XSLT

RESOLVED FIXED in mozilla9

Status

()

Core
XSLT
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Julian Reschke, Assigned: Julian Reschke)

Tracking

({dev-doc-complete})

unspecified
mozilla9
dev-doc-complete
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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

6 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Updated

6 years ago
OS: Windows 7 → All
Hardware: x86 → All
(Assignee)

Comment 1

6 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

6 years ago
Created attachment 551422 [details] [diff] [review]
shrink the accept header for XSLT
Attachment #551422 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #551422 - Flags: review? → review?(jonas)
Attachment #551422 - Flags: review?(jonas) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → julian.reschke
http://hg.mozilla.org/integration/mozilla-inbound/rev/f1820278311e
Keywords: checkin-needed
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/f1820278311e
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
Keywords: dev-doc-needed
(Assignee)

Comment 5

6 years ago
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.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.