[SDCISA-12877] Use Future to replace custom SucceededAsyncResult impl…

… as requested in review
swisspost · Jul 27, 2023 · 15d4b9f · 15d4b9f
1 parent 5f796ac
commit 15d4b9f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 34 deletions.
diff --git a/gateleen-core/src/main/java/org/swisspush/gateleen/core/event/SucceededAsyncResult.java b/gateleen-core/src/main/java/org/swisspush/gateleen/core/event/SucceededAsyncResult.java
diff --git a/gateleen-core/src/main/java/org/swisspush/gateleen/core/http/AbstractHttpClient.java b/gateleen-core/src/main/java/org/swisspush/gateleen/core/http/AbstractHttpClient.java
@@ -6,11 +6,12 @@
 import io.vertx.core.MultiMap;
 import io.vertx.core.Vertx;
 import io.vertx.core.http.*;
-import org.swisspush.gateleen.core.event.SucceededAsyncResult;
 
 import java.util.List;
 import java.util.function.Function;
 
+import static io.vertx.core.Future.succeededFuture;
+
 /**
  * Base class with empty method implementations.
  *
@@ -98,7 +99,7 @@ public Future<HttpClientRequest> request(HttpMethod httpMethod, String s, String
 
     @Override
     public void request(HttpMethod method, String requestURI, Handler<AsyncResult<HttpClientRequest>> handler) {
-        vertx.runOnContext(v -> handler.handle(new SucceededAsyncResult<>(doRequest(method, requestURI))));
+        vertx.runOnContext(v -> succeededFuture(doRequest(method, requestURI)).onComplete(handler));
     }
 
     @Override