Shutdown Vertx as part shutdown hook

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## Next version
+
+### Bugs fixed
+- Ensure that Web3Signer stops the http server when a sigterm is received
+
 ## 24.1.1
 
 This is an optional release for mainnet Ethereum and it includes the updated network configuration for the Sepolia, Holesky and Chiado Deneb forks.

core/src/main/java/tech/pegasys/web3signer/core/Runner.java

@@ -42,6 +42,7 @@
 import java.util.Properties;
 import java.util.StringJoiner;
 import java.util.concurrent.CompletableFuture;
+import java.util.concurrent.CountDownLatch;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeUnit;
 
@@ -174,17 +175,29 @@ public void run() {
 
       persistPortInformation(
           httpServer.actualPort(), metricsService.flatMap(MetricsService::getPort));
+
+      closeables.add(() -> shutdownVertx(vertx));
     } catch (final Throwable e) {
       if (artifactSignerProvider != null) {
         artifactSignerProvider.close();
       }
-      vertx.close();
+      shutdownVertx(vertx);
       metricsService.ifPresent(MetricsService::stop);
       LOG.error("Failed to initialise application", e);
       throw new InitializationException(e);
     }
   }
 
+  private void shutdownVertx(final Vertx vertx) {
+    final CountDownLatch vertxShutdownLatch = new CountDownLatch(1);
+    vertx.close((res) -> vertxShutdownLatch.countDown());
+    try {
+      vertxShutdownLatch.await();
+    } catch (InterruptedException e) {
+      throw new IllegalStateException("Interrupted while waiting for Vertx to stop", e);
+    }
+  }
+
   private MetricsConfiguration createMetricsConfiguration() {
     if (baseConfig.getMetricsPushOptions().isPresent()) {
       MetricsPushOptions options = baseConfig.getMetricsPushOptions().get();