[grid] Closing http client when a session is complete. Fixes #9153

Author: barancev

java/client/src/org/openqa/selenium/remote/tracing/TracedHttpClient.java

@@ -29,6 +29,7 @@
 import org.openqa.selenium.remote.http.HttpResponse;
 import org.openqa.selenium.remote.http.WebSocket;
 
+import java.io.IOException;
 import java.net.URL;
 
 public class TracedHttpClient implements HttpClient {
@@ -58,6 +59,11 @@ public HttpResponse execute(HttpRequest req) {
     }
   }
 
+  @Override
+  public void close() throws IOException {
+    delegate.close();
+  }
+
   public static class Factory implements HttpClient.Factory {
 
     private final Tracer tracer;

java/server/src/org/openqa/selenium/grid/node/config/DriverServiceSessionFactory.java

@@ -42,6 +42,8 @@
 import org.openqa.selenium.remote.tracing.Status;
 import org.openqa.selenium.remote.tracing.Tracer;
 
+import java.io.IOException;
+import java.io.UncheckedIOException;
 import java.net.URI;
 import java.net.URL;
 import java.time.Instant;
@@ -152,6 +154,11 @@ public Optional<ActiveSession> apply(CreateSessionRequest sessionRequest) {
             @Override
             public void stop() {
               service.stop();
+              try {
+                client.close();
+              } catch (IOException e) {
+                throw new UncheckedIOException(e);
+              }
             }
           });
       } catch (Exception e) {