83458 - Update Accept: header

Status: VERIFIED FIXED

(Core :: Networking: HTTP, defect)

Description

[Gervase Markham [:gerv]]

The conclusion in n.p.m.netlib was that the Accept header should be:

Accept: text/xml, application/xml, application/xhtml+xml,
text/html;q=0.9, image/png, image/mng, image/jpeg,
image/gif;q=0.2, text/plain;q=0.8, text/css, */*;q=0.1

(I've tweaked it to save another byte.)

This seems to cover most of the bases while not being unduly long.

Currently, and for 0.9.1, MNG support has regressed due to the imglib2 landing,
so that needs to be left out, leaving:

Accept: text/xml, application/xml, application/xhtml+xml,
text/html;q=0.9, image/png, image/jpeg,
image/gif;q=0.2, text/plain;q=0.8, text/css, */*;q=0.1

Thanks to earlier work, this is now merely a case of updating a pref. It would
be good to check this in before the branch, to save hassle.

Gerv

Comment 1

[Gervase Markham [:gerv]]

Attachment: Accept header update patch

Comment 2

[Gervase Markham [:gerv]]

Looking for r= from [e-mail] or anyone who happens to be trawling the
0.9.1 buglist, rs= and a=from [e-mail] and so on.

I would emphasise that this change is the result of a discussion about the
optimum Accept: header in n.p.m.netlib.

Gerv
(Guardian of the Accept Header)

Comment 3

[Darin Fisher]

r=darin

Comment 4

[Bradley Baetz (:bbaetz)]

r=bbaetz

Comment 5

[Asa Dotzler [:asa]]

a= [e-mail] 

Comment 6

[Gervase Markham [:gerv]]

I never seem to be awake when the tree is open. Any passing kind person has 
my permission to check this in on both trunk and branch, to get it off the 0.9.1 
buglist.

Gerv

Comment 7

[selmer (gone)]

Are either of those r= really rs= or sr=?

Comment 8

[Gervase Markham [:gerv]]

darin is a super-reviewer, so I assumed so. 

For goodness sake, this is a one-line patch. It's not like it needs much poring 
over. If n.p.m.netlib have signed off on it, and Darin has signed off on it, and 
Bradley has signed off on it, and Asa has signed off on it, I think that's 
probably enough oversight.

Gerv

Comment 9

[Gervase Markham [:gerv]]

Checked in on branch and trunk.

Gerv

Comment 10

[Bradley Baetz (:bbaetz)]

VERIFIED via ethereal on linux, and by checking all.js on both trunk and branch
(2001060713)