Closed Bug 590258 Opened 14 years ago Closed 14 years ago

Reformat x86-on-x86_64 part of Linux UA string

Categories

(Core :: Networking: HTTP, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dwitte, Assigned: dwitte)

References

Details

Attachments

(1 file)

Spinoff from bug 589353. We currently have "(X11, Linux x86 (x86_64), rv:...)". The nested parens are guaranteed to cause trouble, and very few people probably test against this particular case. For instance, our very own rv detection algorithm on geckoisgecko.org and https://developer.mozilla.org/en/Browser_Detection_and_Cross_Browser_Support breaks because of this.

We should say "x86 on x86_64" or "x86/x86_64" instead.
Right, the nested parentheses can be confusing, and in fact violate RFC 2616: http://asg.web.cmu.edu/rfc/rfc2616.html#sec-2.2

> "x86 on x86_64" or "x86/x86_64"

The "on" version is unambiguous, though the reverse (x86_64 on x86) unlikely, thus the second form shouldn't be a problem either.
> (correction to comment #1) and in fact violate RFC 2616:

Eh, maybe not - a comment can contain a comment and hence {"(" {"(" ")"} ")"}:

       comment        = "(" *( ctext | quoted-pair | comment ) ")"
       ctext          = <any TEXT excluding "(" and ")">
Whether it violates it isn't relevant here; the fact that it breaks things IRL does.
Agreed.
Attached patch patchSplinter Review
Assignee: nobody → dwitte
Status: NEW → ASSIGNED
Attachment #469257 - Flags: superreview?(jst)
Attachment #469257 - Flags: review?(jst)
Attachment #469257 - Flags: superreview?(jst)
Attachment #469257 - Flags: superreview+
Attachment #469257 - Flags: review?(jst)
Attachment #469257 - Flags: review+
http://hg.mozilla.org/mozilla-central/rev/a0c61c0145ff
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: