Closed Bug 728610 Opened 12 years ago Closed 12 years ago

Zimbra 7.1.3 spits out error 500 if the UA string contains Gecko/13.0a1 or Gecko/13.0.1

Categories

(Tech Evangelism Graveyard :: English US, defect)

defect
Not set
normal

Tracking

(firefox13-)

VERIFIED FIXED
Tracking Status
firefox13 - ---

People

(Reporter: cork, Unassigned)

References

()

Details

(Whiteboard: fixed by bug 728797 / bug 572659)

After the UA string change loading the zimbra webmail fails with a 500 error. No posible workaround, the website crashes before loading the login screen.

Tested with Zimbra 7.1.3_GA_3346
Blocks: 588909
Works: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0a1) Gecko/13 Firefox/13.0a1
Broken: Mozilla/5.0 (Windows NT 6.1; WOW64) Gecko/13.0a1 Firefox/13.0a1
Works: Mozilla/5.0 (Windows NT 6.1; WOW64) Gecko/13 Firefox/13.0a1
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
I'm going to reopen this so we can actually track this for 13.  We may need to back out the UA change.  :(
I don't know if this is related, but my Google search page looks weird now.  No ads, no side bar, it doesn't display the weather if I search for "mountain view weather", etc.
(eg it feels like the google search result page is falling back to a very primitive form because it can't understand the UA string or something)
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee: nobody → english-us
Component: Networking: HTTP → English US
Product: Core → Tech Evangelism
QA Contact: networking.http → english-us
Version: Trunk → unspecified
Summary: Zimbra 7.1.3 crashes in nightly after the UA string change → Zimbra 7.1.3 expects Gecko/[0-9]+ in the UA string, spits out error 500 for Gecko/13.0
Blocks: 723760
Please don't move this out of Core, because the tracking flag will get lost.  Think of this as the "back out the UA change" bug if you want.

Feel free to file a separate evang bug if you think we need one, but note that we have one for Zimbra already (for mail.mozilla.com).
Oh, interesting.  Tracking flag is still there, even though this is in evang?  Leaving it here, then.
Based on comment 1, we could just drop the "a1" part from the UA string to align Nightly and Aurora better with release for sniffing behavior.

FWIW, when I tested various UA string proposals, I thought that Gecko/12.0 was OK, because I tested Zimbra without "a1". I did notice that Zimbra gives error 500 if the character after "Gecko/" is not a digit.
Depends on: 728797
Now http://demo3.zimbra.com/ works for me. Is there any other Web pages to test the breakage?
(In reply to Masatoshi Kimura [:emk] from comment #10)
> Now http://demo3.zimbra.com/ works for me. Is there any other Web pages to
> test the breakage?

https://mail.mozilla.com/
Just +1 from me ... http://demo3.zimbra.com/ is broken as well as our internal Zimbra web user interface (which is Zimbra 7.1.2_GA_3268 (build 20110804120319)).
Summary: Zimbra 7.1.3 expects Gecko/[0-9]+ in the UA string, spits out error 500 for Gecko/13.0 → Zimbra 7.1.3 spits out error 500 if the UA string contains Gecko/13.0a1 or Gecko/13.0.1
(In reply to Matej Cepl from comment #13)
> Just +1 from me ... http://demo3.zimbra.com/ is broken as well as our
> internal Zimbra web user interface (which is Zimbra 7.1.2_GA_3268 (build
> 20110804120319)).

and yes with US

Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/13.0a1 Firefox/13.0a1

both websites work for me again.
Matej: are you sure? That looks like the UA that everyone says is breaking things. Just to be crystal clear, can you post a broken UA and a working UA?

Gerv
(In reply to Gervase Markham [:gerv] from comment #15)
> Matej: are you sure? That looks like the UA that everyone says is breaking
> things. Just to be crystal clear, can you post a broken UA and a working UA?

No, it isn't the same.

The difference is

Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/13.0a1 Firefox/13.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:13) Gecko/13.0a1 Firefox/13.0a1
(In reply to Gervase Markham [:gerv] from comment #15)
> Matej: are you sure? That looks like the UA that everyone says is breaking
> things. Just to be crystal clear, can you post a broken UA and a working UA?

Yes, there was a typo in comment 14, that was the original unchanged UA. With the changes indicated in comment 16 it really works.
Matej: I'm still not clear, sorry. The two UAs you give differ in the rv: token; I thought the breakage was because we changed the Gecko token?

Please fill in this form:

Broken: <insert UA here>
Working: <insert UA here>

:-)

Gerv
For me its:
Broken:
Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/13.0a1 Firefox/13.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/13.0a1 Firefox/13.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:13) Gecko/13.0a1 Firefox/13.0a1

Works:
Mozilla/5.0 (X11; Linux x86_64; rv:13) Gecko/20100101 Firefox/13.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/20100101 Firefox/13.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:13.0a1) Gecko/13.0 Firefox/13.0a1

So ya its definitely the Gecko/ that affects the server.
Awesome. Then we are on the right track with our fix.

Gerv
Status: REOPENED → RESOLVED
Closed: 12 years ago12 years ago
Resolution: --- → FIXED
Whiteboard: fixed by bug 728797 / bug 572659
I can confirm that the respun nightly (20120220074932, which contains the fix from bug 572659) fixes this issue. Both http://demo3.zimbra.com/ and https://mail.mozilla.com/ gave 500 errors in the first nightly but show login pages in this build.
Status: RESOLVED → VERIFIED
Thank you, today's nightly works correctly for me as well with our company Zimbra.
We're using bug 588909 to track the backout and steps forward. FF13 should no longer be affected now that the backout has been performed.
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.