Only log at DEBUG if graphql endpoint already finished the HTTP response

quarkusio · Feb 13, 2023 · 4252cef · 4252cef
1 parent 96203bb
commit 4252cef
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/...me/src/main/java/io/quarkus/smallrye/graphql/runtime/SmallRyeGraphQLExecutionHandler.java b/...me/src/main/java/io/quarkus/smallrye/graphql/runtime/SmallRyeGraphQLExecutionHandler.java
@@ -339,9 +339,16 @@ public void write(ExecutionResponse er) {
                 response.setStatusCode(500)
                         .end();
             } else {
-                response.setStatusCode(200)
-                        .setStatusMessage(OK)
-                        .end(Buffer.buffer(er.getExecutionResultAsString(), requestedCharset));
+                try {
+                    response.setStatusCode(200)
+                            .setStatusMessage(OK)
+                            .end(Buffer.buffer(er.getExecutionResultAsString(), requestedCharset));
+                } catch (IllegalStateException ise) {
+                    // The application already finished the request by itself for some reason
+                    if (log.isDebugEnabled()) {
+                        log.debug("Cannot write response", ise);
+                    }
+                }
             }
         }