311263 - CalDAV provider should send a proper 'accept' header

Status: RESOLVED FIXED

(Calendar :: Provider: CalDAV, defect)

Description

[Helge Hess]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b4) Gecko/20050908 Firefox/1.4
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b4) Gecko/20050908 Firefox/1.4

When issuing CalDAV REPORT requests the Sunbird CalDAV provider sends such
"Accept" headers:
---snip---
Accept:
text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5
---snap---

Since a CalDAV report is always XML and Sunbird can't deal with anything else,
it should be just:
---snip---
Accept: text/xml
---snap---


In the same run it would be nice to change the default charset from:
---snip---
Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
---snap---
to:
---snip---
Accept-Charset: utf-8,*;q=0.1
---snap---
Latin-1 as the preferred charset is really inappropriate and might trigger
unnecessary server-side charset conversions to please the client ...

Note: the latter is no real bug, just a subminor annoyance.


Reproducible: Always

Steps to Reproduce:
1. subscribe a remote calendar in Sunbird and trace the HTTP interaction

Actual Results:  
Accept:
text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,*/*;q=0.5


Expected Results:  
Accept: text/xml

Comment 1

[Bruno Browning]

Attachment: set request headers

this addresses bug 311265 as well, and possibly bug 308569 as well, though the  latter is not clear to me.

Comment 2

[cmtalbert]

Comment on attachment 245927 [details] [diff] [review]
set request headers

This looks good. The only nit I have is that on the first webSvc.PutFromString line (the 295 block) there is a space at the end of the line. Remove that and r+

Comment 3

[Matthew (lilmatt) Willis]

Moving from dmose to Bruno since the former's on vacation and the latter's the one working on it.  Might as well confirm it also. :)

Comment 4

[Bruno Browning]

Attachment: correction per reviewer

Comment 5

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 246240 [details] [diff] [review]
correction per reviewer

r2=mvl

Comment 6

[Matthew (lilmatt) Willis]

Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED