Last Comment Bug 656251 - reduce verbosity of Accept header for fetching XSLT
: reduce verbosity of Accept header for fetching XSLT
Status: RESOLVED FIXED
: dev-doc-complete
Product: Core
Classification: Components
Component: XSLT (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla9
Assigned To: Julian Reschke
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-05-11 04:45 PDT by Julian Reschke
Modified: 2011-11-04 12:11 PDT (History)
5 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
shrink the accept header for XSLT (927 bytes, patch)
2011-08-08 04:52 PDT, Julian Reschke
jonas: review+
Details | Diff | Splinter Review

Description Julian Reschke 2011-05-11 04:45:46 PDT
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
Comment 1 Julian Reschke 2011-08-08 04:50:07 PDT
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
Comment 2 Julian Reschke 2011-08-08 04:52:39 PDT
Created attachment 551422 [details] [diff] [review]
shrink the accept header for XSLT
Comment 3 Phil Ringnalda (:philor) 2011-09-10 21:20:04 PDT
http://hg.mozilla.org/integration/mozilla-inbound/rev/f1820278311e
Comment 4 Matt Brubeck (:mbrubeck) 2011-09-11 06:35:33 PDT
http://hg.mozilla.org/mozilla-central/rev/f1820278311e
Comment 5 Julian Reschke 2011-09-19 10:38:51 PDT
I don't believe this requires any developer-visible documentation.
Comment 6 Eric Shepherd [:sheppy] 2011-11-04 12:11:36 PDT
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.

Note You need to log in before you can comment on or make changes to this bug.