My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 1694: Review via REST returning 500s
1 person starred this issue and may be notified of changes. Back to list
Status:  Released
Owner:  sop@google.com
Closed:  Nov 2012


Sign in to add a comment
 
Reported by cco3@android.com, Nov 29, 2012
curl -s https://android-review.googlesource.com/a/changes/platform/packages/apps/Exchange,master,Id5d8b8ec4e0349c86edfbe1d0f58d50d9146cf43/revisions/44b7f750360af040fefe767b4a2efcd52434214d/review -H "Authorization: OAuth <myoauth>" -d '{"labels": {"Verified": 0}}' -H "Content-Type: application/json" | html2text

[Google]
500. That???s an error.
There was an error. Please try again later. That???s all we know.
Nov 29, 2012
#1 cco3@android.com
If I change a character in my OAuth, I get 403, so I think I'm authenticating correctly.

Gerrit version: 2.5-638-g62a5324
Nov 29, 2012
#2 sop@google.com
s,/,%2F,g in the project name. It can't parse the URL you fed it.
Nov 29, 2012
#4 sop@google.com
om.google.inject.ProvisionException: Guice provision errors:

1) null returned by binding at com.google.gerrit.server.change.EmailReviewComments$Factory.create()
 but parameter 8 of com.google.gerrit.server.change.EmailReviewComments.<init>() is not @Nullable
  while locating com.google.gerrit.reviewdb.client.ChangeMessage annotated with @com.google.inject.assistedinject.Assisted(value=)
    for parameter 8 at com.google.gerrit.server.change.EmailReviewComments.<init>(EmailReviewComments.java:77)
  while locating com.google.gerrit.server.change.EmailReviewComments annotated with interface com.google.inject.assistedinject.Assisted

1 error
	at com.google.inject.internal.InjectorImpl$3.get(InjectorImpl.java:1011)
	at com.google.inject.assistedinject.FactoryProvider2.invoke(FactoryProvider2.java:632)
	at com.google.gerrit.server.change.$Proxy89.create(Unknown Source)
	at com.google.gerrit.server.change.PostReview.apply(PostReview.java:154)

Status: Accepted
Owner: sop@google.com
Nov 29, 2012
#5 sop@google.com
I think was fixed by be43be64. Unfortunately its not yet deployed on our server.
Nov 29, 2012
#6 sop@google.com
be43be64 deployed for MTV folks. Rest of the world shortly.
Status: Released
Sign in to add a comment

Powered by Google Project Hosting