Issue 226: Internal server error
Status:  Released
Owner:
Closed:  Oct 2012
Reported by code-rev...@gtempaccount.com, Sep 24, 2009
Reported by Leon Scroggins III <scroggo@google.com> on Thu Jun 18 07:51:16 PDT 2009
Source: JIRA GERRIT-227
Environment: windows XP, chrome

Go to https://android-git.corp.google.com/g/Gerrit#patch,sidebyside,4605,2,WebKit/android/jni/WebViewCore.cpp
Click patch history
Click the radio button in the old version column, row Patch Set 1 .

I get a dialog saying:
Code Review - Unexpected Error
Server Error
Internal Server Error

with the option to close the dialog.  After closing the dialog, the page is
still active and works, but I cannot see the diff between Patch Set 1 and
Patch Set 2.  By speaking with the person who uploaded the change, I am aware
that the two versions should be identical.
Sep 24, 2009
#1 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Thu Jun 18 07:53:14 PDT 2009

Looks like the server side stack trace is:

2009-06-18 07:44:01.301:/g:WARN:  Error in patchScript
java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
	at java.util.ArrayList.RangeCheck(ArrayList.java:547)
	at java.util.ArrayList.get(ArrayList.java:322)
	at com.google.gerrit.server.patch.PatchScriptBuilder.packContent
(PatchScriptBuilder.java:317)
	at com.google.gerrit.server.patch.PatchScriptBuilder.toPatchScript
(PatchScriptBuilder.java:139)
	at com.google.gerrit.server.patch.PatchScriptAction.run
(PatchScriptAction.java:133)
	at com.google.gerrit.server.patch.PatchScriptAction.run
(PatchScriptAction.java:56)
	at com.google.gerrit.client.rpc.BaseServiceImplementation.run
(BaseServiceImplementation.java:48)
	at com.google.gerrit.server.patch.PatchDetailServiceImpl.patchScript
(PatchDetailServiceImpl.java:85)
Sep 24, 2009
#2 code-rev...@gtempaccount.com
Comment by Shawn Pearce <sop@google.com> on Thu Jun 18 16:35:41 PDT 2009

Fixed by https://review.source.android.com/10464
Sep 24, 2009
#3 code-rev...@gtempaccount.com
Update by Shawn Pearce <sop@google.com> on Thu Jun 18 16:35:41 PDT 2009

Fixed in version 2.0.15.
Status: Fixed
Sep 25, 2009
#4 code-rev...@gtempaccount.com
(No comment was entered for this change.)
Labels: FixedIn-2.0.15
Oct 25, 2012
#5 sop@google.com
(No comment was entered for this change.)
Status: Released