Merge pull request #20787 from stuartwdouglas/20778

Don't attempt to end the request if already ended
quarkusio · Oct 15, 2021 · 09faf5d · 09faf5d
2 parents e6c3cee + 63d9059
commit 09faf5d
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/...rtx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/security/HttpAuthorizer.java b/...rtx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/security/HttpAuthorizer.java
@@ -1,5 +1,6 @@
 package io.quarkus.vertx.http.runtime.security;
 
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.function.BiFunction;
@@ -170,12 +171,20 @@ public void onSubscribe(UniSubscription subscription) {
 
                         @Override
                         public void onItem(Boolean item) {
-                            routingContext.response().end();
+                            if (!routingContext.response().ended()) {
+                                routingContext.response().end();
+                            }
                         }
 
                         @Override
                         public void onFailure(Throwable failure) {
-                            routingContext.fail(failure);
+                            if (!routingContext.response().ended()) {
+                                routingContext.fail(failure);
+                            } else if (!(failure instanceof IOException)) {
+                                log.error("Failed to send challenge", failure);
+                            } else {
+                                log.debug("Failed to send challenge", failure);
+                            }
                         }
                     });
                 } else {