Closed Bug 80020 Opened 23 years ago Closed 23 years ago

Accept_Language and Accept_Charset format error

Categories

(Core :: Internationalization, defect, P2)

defect

Tracking

()

VERIFIED FIXED
mozilla0.9.2

People

(Reporter: benoit, Assigned: darin.moz)

References

()

Details

(Keywords: intl, Whiteboard: r=bbaetz, sr=darin, a=asa)

Attachments

(3 files)

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.2.18 i686; en-US; rv:0.9) Gecko/20010505
BuildID:    2001050921

Depending on the locale, Accept_Language and Accept_Charset have an incorrect
format regarding the quality attribute. With a fr_FR locale, i have
Accept-Language: fr, en-us; q=0,750, en; q=0,500, nl; q=0,250. The correct
format would be Accept-Language: fr, en-us; q=0.750, en; q=0.500, nl; q=0.250
(decimal POINT rather than the locale COMMA).

Reproducible: Always
Steps to Reproduce:
1. Set your locale to fr_FR (on Linux/bash I do 'export LANG=fr_FR').
2. Start Mozilla (within the shell where you changed the locale)
3. Make sure you have at least 2 accepted languages in
Edit/Preferences/Navigator/Languages.
4. Start an HTTP request, for example to my demonstration page
http://milkyweb.com/mozilla/h.php


Actual Results:  The Web server receives the HTTP headers wrongly formatted, as in:
Accept-Language: fr, en-us; q=0,750, en; q=0,500, nl; q=0,250

Expected Results:  The right format:
Accept-Language: fr, en-us; q=0.750, en; q=0.500, nl; q=0.250

Reference : RFC 2068, HTTP/1.1 page 97-99.
I'm using Linux/Suse 7.1.
Marking NEW this is also sen in Mail/NEws X-Acceptted Languages as per bug 66734
which i am marking as a dupe.
Assignee: neeti → nhotta
Status: UNCONFIRMED → NEW
Component: Networking: HTTP → Internationalization
Ever confirmed: true
Keywords: intl, ui
QA Contact: tever → andreasb
*** Bug 66734 has been marked as a duplicate of this bug. ***
As this is now assigned to the Internationalization component, I would like to
add a comment:
I think the I18N does its job well. It's just that these 2 HTTP headers (and
possibly all headers, but I'm not sure) should not be localized at all. They
should follow the HTTP protocol syntax.
Well, you know better than me which component to assign to, but i wanted to be
clear.

Another note: I tested this with Mozilla 0.9 on WinNT4 SP6 and the headers are
correctly formatted with my locale settings.
Adding momoi to cc who knows about this very well.
Momoi san, any change around this area since NS6?
QA Contact: andreasb → ylong
NS6 (and Mozilla 0.8 if i remember well) did not set the quality value on
accepted languages.

Here's what i have with NS6 (Mozilla/5.0 (X11; U; Linux 2.2.14-6.1.1smp i686;
en-US; m18) Gecko/20001107 Netscape6/6.0):

Accept-Language: fr, en
Accept-Charset: 

(using my example URL above)
Reassign to havill@redhat.com.
Assignee: nhotta → havill
The Q-values decimal point is indeed susceptible to 
LC_NUMERIC value differences. (5/15/2001 Linux build).
I don't think this is an localization issue. The problem
seems to be an i18n one. Other HTTP headers sent, e.g.
User_Agent string does not have this problem under
fr_FR locale.
The following patch should fix the comma/dot I18N issue with the HTTP protocol.
As this affects the protocol, this should be fixed ASAP.

I need a r= and sr= please.
Severity: normal → major
Status: NEW → ASSIGNED
Ugh! Missed one character in the last patch! I blame the keyboard. Use the next
patch.

need a sr= and r=, and then someone to check it in please.

Verify the output with:

http://www.mozilla.gr.jp:4321/
this looks good to me (sr=darin), but while you're at it why not reduce the
precision?  do we really need 3 decimal places?
Darin Fisher wrote:
> this looks good to me (sr=darin), but while you're at it why not reduce the
> precision?  do we really need 3 decimal places?

Not really, unless the user adds more than 99 languages (and even if they do
this, protocol wise the sent request will still be syntax-correct). ;-P. Will
reduce to 2 decimal places. Please check in the next patch.
OS: Linux → All
Hardware: PC → All
sr=darin
-> me, so i can get this checked in.
Assignee: havill → darin
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
Keywords: patch
Target Milestone: --- → mozilla0.9.2
Priority: -- → P2
r=bbaetz
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)
Blocks: 83989
Whiteboard: r=bbaetz, sr=darin, a=asa
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified it shows decimal point(dot) not comma on 06-13 trunk build.
Status: RESOLVED → VERIFIED
*** Bug 77700 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: