commit 49d4ab881262cad10d684eee8af9350371df1f11
author Jacek Centkowski <jcentkowski@collab.net> Tue Sep 19 14:21:42 2017 +0200
committer Jacek Centkowski <jcentkowski@collab.net> Tue Sep 19 14:22:04 2017 +0200
tree 2fc2b675047690e11db134c02e73f6c44e0c9ed8
parent 56dabb4f34385c007ee9bdcebcd35953fe5b9085

Add 201 (CREATED) status to successful HTTP responses

This is accepted response to POST method.

Change-Id: I4a188e373591616e495825f3abe094c3c192b837
Signed-off-by: Jacek Centkowski <jcentkowski@collab.net>

src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java

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 994d337..0c23b6e 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/webhooks/HttpResponseHandler.java
@@ -14,6 +14,7 @@
 
 package com.googlesource.gerrit.plugins.webhooks;
 
+import static javax.servlet.http.HttpServletResponse.SC_CREATED;
 import static javax.servlet.http.HttpServletResponse.SC_NO_CONTENT;
 import static javax.servlet.http.HttpServletResponse.SC_OK;
 
@@ -47,7 +48,7 @@
 
   private boolean isSuccessful(HttpResponse response) {
     int sc = response.getStatusLine().getStatusCode();
-    return sc == SC_NO_CONTENT || sc == SC_OK;
+    return sc == SC_CREATED || sc == SC_NO_CONTENT || sc == SC_OK;
   }
 
   private String parseResponse(HttpResponse response) {