| Issue 1694: | Review via REST returning 500s | |
| 1 person starred this issue and may be notified of changes. | Back to list |
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
Nov 29, 2012
s,/,%2F,g in the project name. It can't parse the URL you fed it.
Nov 29, 2012
Shouldn't it 400 over something like that? URL changed. Still getting 500s. https://android-review.googlesource.com/a/changes/platform%2Fpackages%2Fapps%2FExchange~master~Id5d8b8ec4e0349c86edfbe1d0f58d50d9146cf43/revisions/44b7f750360af040fefe767b4a2efcd52434214d/review
Nov 29, 2012
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
I think was fixed by be43be64. Unfortunately its not yet deployed on our server.
Nov 29, 2012
be43be64 deployed for MTV folks. Rest of the world shortly.
Status:
Released
|
|
| ► Sign in to add a comment |