202918 - TM in new Firebird UA string causes 400 bad request server error

Status: VERIFIED FIXED

(Firefox :: General, defect)

Description

[Bill Mason]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4a) Gecko/20030422 Firebird™ Browser/0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4a) Gecko/20030422 Firebird™ Browser/0.6

This server returns error 400 when visited.  Using the uabar extension
(http://uabar.mozdev.org/) I removed the TM from the current Firebird UA string.
 The site then gave me the page.

Reproducible: Always

Steps to Reproduce:
1. Visit http://www.fox.com/mrpersonality/, get error 400.
2. Remove TM from the UA string and revisit. Page renders normally.

Actual Results:  
Server returns error 400.

Expected Results:  
Page should be presented normally.

The page works fine in the prev Phoenix nightly 2030411 and in Mozilla 1.4b
nightlies.

Comment 1

[Bill Mason]

Fixing to correct component.

Comment 2

[David Tenser [:djst]]

This one is yours Ben. ;)

Comment 3

[Bill Mason]

Another URL with the same problem (and workaround):
http://tv.zap2it.com/news/tvnewsdaily_headlines.html

Comment 4

[Sébastien Delahaye]

*** Bug 202919 has been marked as a duplicate of this bug. ***

Comment 5

[Sébastien Delahaye]

*** Bug 202926 has been marked as a duplicate of this bug. ***

Comment 6

[Gennady Feldman]

Ok I got a quick patch that should easily fix the problem. Just edit all.js file
and change the line 50 to:

 pref("general.useragent.vendor", "Firebird Browser");

I just removed the TM symbol from that line. Should I attach it as a proposed fix?

Btw, this does fix the loading problems for some of the websites that are
reporting loading issues. [See bug 202919]

Comment 7

[Bill Mason]

Attachment: Revised all.js to change the TM symbol

Given all the legal hand-wringling of late over Firebird, just eliminating it
from the UA string might not be best.  Someone thought it important enough to
put there in the first place.

This revised all.js replaces the current trademark symbol with (TM) instead.

Comment 8

[alanjstr]

Bill-
Please create a diff instead of including the whole all.js file.  

And I suggest Sev -> Major

Comment 9

[Simon Paquet [:sipaq]]

Increasing severity. This is a definite blocker.

Comment 10

[noririty]

Thanks.
Fix checked in by Ben.

Comment 11

[Bill Mason]

*** Bug 203055 has been marked as a duplicate of this bug. ***

Comment 12

[Sébastien Delahaye]

v

Comment 13

[Sébastien Delahaye]

*** Bug 203113 has been marked as a duplicate of this bug. ***

Comment 14

[noririty]

*** Bug 203126 has been marked as a duplicate of this bug. ***

Comment 15

[Boris Zbarsky [:bzbarsky]]

*** Bug 203130 has been marked as a duplicate of this bug. ***

Comment 16

[Sébastien Delahaye]

*** Bug 203241 has been marked as a duplicate of this bug. ***

Comment 17

[Boris Zbarsky [:bzbarsky]]

*** Bug 203240 has been marked as a duplicate of this bug. ***

Comment 18

[Sébastien Delahaye]

*** Bug 203276 has been marked as a duplicate of this bug. ***

Comment 19

[noririty]

*** Bug 203788 has been marked as a duplicate of this bug. ***

Comment 20

[Sébastien Delahaye]

*** Bug 203962 has been marked as a duplicate of this bug. ***