Don't always recreate the http client

Author: Lenni0451

src/main/java/net/raphimc/authhook/AuthHookHttpServer.java

@@ -34,13 +34,16 @@
 import java.net.http.HttpClient;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
 import java.util.concurrent.TimeUnit;
 
 public class AuthHookHttpServer {
 
     private final InetSocketAddress bindAddress;
     private final ChannelFuture channelFuture;
     private final Map<String, ProxyConnection> pendingConnections = CacheBuilder.newBuilder().expireAfterWrite(1, TimeUnit.MINUTES).<String, ProxyConnection>build().asMap();
+    private final HttpClient httpClient = HttpClient.newBuilder().executor(Executors.newCachedThreadPool()).build();
 
     public AuthHookHttpServer(final InetSocketAddress bindAddress) {
         this.bindAddress = bindAddress;
@@ -100,7 +103,6 @@ protected void channelRead0(ChannelHandlerContext ctx, Object msg) {
                                         }
                                     }
 
-                                    final HttpClient httpClient = HttpClient.newHttpClient();
                                     httpClient.sendAsync(java.net.http.HttpRequest.newBuilder().uri(URI.create("https://sessionserver.mojang.com" + uri)).build(), java.net.http.HttpResponse.BodyHandlers.ofByteArray())
                                             .thenAccept(response -> {
                                                 final FullHttpResponse fullHttpResponse = new DefaultFullHttpResponse(HttpVersion.HTTP_1_1, HttpResponseStatus.valueOf(response.statusCode()), ctx.alloc().buffer());
@@ -137,6 +139,13 @@ public void stop() {
         if (this.channelFuture != null) {
             this.channelFuture.channel().close();
         }
+        this.httpClient.executor().map(ExecutorService.class::cast).ifPresent(ExecutorService::shutdown);
+        if (this.httpClient instanceof AutoCloseable closeable) {
+            try {
+                closeable.close();
+            } catch (Exception ignored) {
+            }
+        }
     }
 
     public Channel getChannel() {