Accept also 200 OK as a success response from a webhooks target
Change-Id: I4bcbd5cf839f768977844119d135e1dc5e534439
diff --git a/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java b/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java
index 73d9f28..8654eed 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java
@@ -15,6 +15,7 @@
package com.googlesource.gerrit.plugins.webhooks;
import static javax.servlet.http.HttpServletResponse.SC_NO_CONTENT;
+import static javax.servlet.http.HttpServletResponse.SC_OK;
import com.googlesource.gerrit.plugins.webhooks.HttpResponseHandler.HttpResult;
@@ -48,7 +49,8 @@
}
private boolean isSuccessful(HttpResponse response) {
- return response.getStatusLine().getStatusCode() == SC_NO_CONTENT;
+ int sc = response.getStatusLine().getStatusCode();
+ return sc == SC_NO_CONTENT || sc == SC_OK;
}
private String parseResponse(HttpResponse response) {
