[openhabcloud] Accept all Jetty supported http method types (openhab#…

…10600) * Accept all Jetty supported http method types This fixes a number of open issues due to the fact that we were only accepting a limited number of http method types. The effect of this was some functionality like DELETE or HEAD requests would just not work when using the cloud service, which madee our UI look broken in different ways, also it poluted the users log with a lot of messages. Fixes openhab/openhab-core#2312 Fixes openhab/openhab-cloud#328 Signed-off-by: Dan Cunningham <[email protected]>
computergeek1507 · Jul 13, 2021 · 7d7ccdc · 7d7ccdc
1 parent 0f43d75
commit 7d7ccdc
Showing 1 changed file with 7 additions and 12 deletions.
diff --git a/...enhab.io.openhabcloud/src/main/java/org/openhab/io/openhabcloud/internal/CloudClient.java b/...enhab.io.openhabcloud/src/main/java/org/openhab/io/openhabcloud/internal/CloudClient.java
@@ -315,20 +315,15 @@ private void handleRequestEvent(JSONObject data) {
                 proto = data.getString("protocol");
             }
             request.header("X-Forwarded-Proto", proto);
-
-            if (requestMethod.equals("GET")) {
-                request.method(HttpMethod.GET);
-            } else if (requestMethod.equals("POST")) {
-                request.method(HttpMethod.POST);
-                request.content(new BytesContentProvider(requestBody.getBytes()));
-            } else if (requestMethod.equals("PUT")) {
-                request.method(HttpMethod.PUT);
-                request.content(new BytesContentProvider(requestBody.getBytes()));
-            } else {
-                // TODO: Reject unsupported methods
-                logger.warn("Unsupported request method {}", requestMethod);
+            HttpMethod method = HttpMethod.fromString(requestMethod);
+            if (method == null) {
+                logger.debug("Unsupported request method {}", requestMethod);
                 return;
             }
+            request.method(method);
+            if (!requestBody.isEmpty()) {
+                request.content(new BytesContentProvider(requestBody.getBytes()));
+            }
 
             request.onResponseHeaders(response -> {
                 logger.debug("onHeaders {}", requestId);