Merge pull request #21295 from famod/fix-both-random-ports

Fix IllegalArgumentException in rest-assured if http and https ports are both random
quarkusio · Nov 9, 2021 · e869680 · e869680
2 parents bbe806e + f81a64c
commit e869680
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...ons/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/VertxHttpRecorder.java b/...ons/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/runtime/VertxHttpRecorder.java
@@ -713,7 +713,8 @@ private static HttpServerOptions createSslOptions(HttpBuildTimeConfig buildTimeC
         serverOptions.setSni(sslConfig.sni);
         serverOptions.setHost(httpConfiguration.host);
         int sslPort = httpConfiguration.determineSslPort(launchMode);
-        serverOptions.setPort(sslPort == 0 ? -1 : sslPort);
+        // -2 instead of -1 (see http) to have vert.x assign two different random ports if both http and https shall be random
+        serverOptions.setPort(sslPort == 0 ? -2 : sslPort);
         serverOptions.setClientAuth(buildTimeConfig.tlsClientAuth);
         serverOptions.setReusePort(httpConfiguration.soReusePort);
         serverOptions.setTcpQuickAck(httpConfiguration.tcpQuickAck);