Issue 40: Gerrit doesn't work in Opera - Unexpected Error - StatusCodeException
Status:  Released
Owner:
Closed:  Oct 2012
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Shawn Pearce <sop@google.com> on Tue Jan 27 07:08:49 PST 2009
Source: JIRA GERRIT-40
Affected Version: 2.0

Cannot access review.source.android.com with Opera

An error dialog pops up:

Code Review - Unexpected Error
com.google.gwt.user.client.rpc.StatusCodeException
Bad Request
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Tue Jan 27 07:45:45 PST 2009

gwtjsonrpc assumes the browser will send "Accept: application/json", because
that is what the client side JavaScript set into the AJAX request.

Opera appends on a lot of other MIME types:

"Accept: application/json, text/html, application/xml;q=0.9, application/xhtml
+xml, image/png, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1"

Resulting in the server aborting the transaction and sending "400 Bad Request"
to the client.

I'll have to modify the way I'm testing the Accept header in JsonServlet and
spin a new gwtjsonrpc release.
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Tue Jan 27 08:10:24 PST 2009

Fixed by http://review.source.android.com/8666.  Patch needs to be rolled to
production still.
Sep 24, 2009
#3 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Tue Jan 27 10:58:06 PST 2009

Fixed in version 2.0.1.
Sep 24, 2009
#4 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Tue Jan 27 14:32:22 PST 2009
Status: Fixed
Sep 25, 2009
#5 code-rev...@gtempaccount.com
(No comment was entered for this change.)
Labels: FixedIn-2.0.1
Mar 11, 2011
#6 pchid...@gmail.com
I get this when I try to access the review from my opera browser. Is this an servlet issue as well?

JavaScript - https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html
Unknown thread
Uncaught exception: TypeError: 'a.prettyPrintOne' is not a function
Error thrown at line 378, column 39 in nMb(a, b, c, d) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    return a.prettyPrintOne(b,c)
called from line 321, column 20 in lMb(a, b, c) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    return nMb($wnd[iMb.c],b,c,a.f.m)
called from line 543, column 139 in ZLb(a, b) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    c=lMb(a,c,$Lb(a))
called from line 430, column 103 in $Gb(a) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    ZLb(c,a.c);
called from line 551, column 48 in llb(a) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    b=$Gb(a);
called from line 297, column 42 in qdb(a, b, c, d, e) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    a.wc(e)
called from line 544, column 533 in wib(a, b, c) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    qdb(a.f,a.q,a.j,a.k,b);
called from line 341, column 16 in xjb(a) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    this.c.u==this.e&&wib(this.c,a,this.d)
called from line 123, column 30 in wG(a) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    this.Rb(a);
called from line 167, column 130 in ofd(b, c, d) in https://review.source.android.com/gerrit/366DE73696967BD9C9840AD4C943A5F8.cache.html:
    c.yb(g)

Oct 25, 2012
#7 sop@google.com
(No comment was entered for this change.)
Status: Released