killbill-memoizeit

jaxrs: fix confusing JAX-RS log line displaying wrong HTTP

6/8/2018 5:26:38 PM

Details

diff --git a/jaxrs/src/main/java/org/killbill/billing/jaxrs/resources/PluginResource.java b/jaxrs/src/main/java/org/killbill/billing/jaxrs/resources/PluginResource.java
index 27bf057..e6eb326 100644
--- a/jaxrs/src/main/java/org/killbill/billing/jaxrs/resources/PluginResource.java
+++ b/jaxrs/src/main/java/org/killbill/billing/jaxrs/resources/PluginResource.java
@@ -190,11 +190,9 @@ public class PluginResource extends JaxRsResourceBase {
             if (response.getStatus() >= 400) {
                 log.warn("{} responded {}", request.getPathInfo(), response.getStatus());
             }
-            // Jersey will want to return 204, but the servlet should have done the right thing already
-            return null;
-        } else {
-            return Response.status(response.getStatus()).build();
         }
+
+        return Response.status(response.getStatus()).build();
     }
 
     private InputStream createInputStream(final HttpServletRequest request, final MultivaluedMap<String, String> form) throws IOException {