use the default HttpClient executor in DependencyDownloader

(HttpClient *really* doesn't want to be used synchronously)

by reusing the HttpClient we still avoid redundant threads, we just can't reuse the thread making the request for handling it

common/src/main/java/net/draycia/carbon/common/util/DependencyDownloader.java

@@ -67,10 +67,7 @@ public DependencyDownloader(
     ) {
         this.logger = logger;
         this.cacheDir = cacheDir;
-        this.client = HttpClient.newBuilder()
-            // using a cached thread pool (default) is completely redundant with how we use the client
-            .executor(Runnable::run)
-            .build();
+        this.client = HttpClient.newHttpClient();
     }
 
     public Set<Path> resolve() {