added new tests

Author: iampopovich

java/src/org/openqa/selenium/remote/http/jdk/JdkHttpClient.java

@@ -80,10 +80,12 @@ public class JdkHttpClient implements HttpClient {
   private final ExecutorService executorService;
   private final Duration readTimeout;
   private final Duration connectTimeout;
+  private final ClientConfig config;
 
   JdkHttpClient(ClientConfig config) {
     Objects.requireNonNull(config, "Client config must be set");
-
+    
+    this.config = config;
     this.messages = new JdkHttpMessages(config);
     this.readTimeout = config.readTimeout();
     this.connectTimeout = config.connectionTimeout();
@@ -338,7 +340,7 @@ public WebSocket send(Message message) {
                 throw new WebDriverException(cause);
               } catch (InterruptedException e) {
                 Thread.currentThread().interrupt();
-                throw new WebDriverException(e.getMessage());
+                throw new WebDriverException(e.getMessage(), e);
               } catch (java.util.concurrent.TimeoutException e) {
                 throw new TimeoutException(e);
               } finally {
@@ -522,6 +524,11 @@ private HttpResponse execute0(HttpRequest req) throws UncheckedIOException {
     }
   }
 
+  // Package-private method for testing
+  URI getBaseUri() {
+    return config.baseUri();
+  }
+
   @Override
   public void close() {
     if (this.client == null) {

java/test/org/openqa/selenium/remote/http/jdk/JdkHttpClientTest.java

@@ -19,11 +19,65 @@
 
 import org.openqa.selenium.remote.http.HttpClient;
 import org.openqa.selenium.remote.internal.HttpClientTestBase;
+import org.junit.Test;
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.isNull;
+
+import java.net.URI;
+import java.net.URISyntaxException;
 
 class JdkHttpClientTest extends HttpClientTestBase {
 
   @Override
   protected HttpClient.Factory createFactory() {
     return new JdkHttpClient.Factory();
   }
+
+  @Test
+  void shouldStripCredentialsFromUrl() throws URISyntaxException {
+    URI originalUri = new URI("http://admin:password@localhost:4444/wd/hub");
+    ClientConfig config = ClientConfig.defaultConfig().baseUri(originalUri);
+    
+    JdkHttpClient client = new JdkHttpClient(config);
+    
+    // Get the modified URI from the client's config
+    URI modifiedUri = client.getBaseUri();
+    
+    assertThat(modifiedUri.getUserInfo()).isNull();
+    assertThat(modifiedUri.getHost()).isEqualTo("localhost");
+    assertThat(modifiedUri.getPort()).isEqualTo(4444);
+    assertThat(modifiedUri.getPath()).isEqualTo("/wd/hub");
+  }
+
+  @Test
+  void shouldHandleUrlWithoutCredentials() throws URISyntaxException {
+    URI originalUri = new URI("http://localhost:4444/wd/hub");
+    ClientConfig config = ClientConfig.defaultConfig().baseUri(originalUri);
+    
+    JdkHttpClient client = new JdkHttpClient(config);
+    
+    URI modifiedUri = client.getBaseUri();
+    
+    assertThat(modifiedUri).isEqualTo(originalUri);
+  }
+
+  @Test
+  void shouldPreserveUrlComponentsExceptCredentials() throws URISyntaxException {
+    URI originalUri = new URI(
+        "https://admin:password@localhost:4444/wd/hub?debug=true#fragment");
+    ClientConfig config = ClientConfig.defaultConfig().baseUri(originalUri);
+    
+    JdkHttpClient client = new JdkHttpClient(config);
+    
+    URI modifiedUri = client.getBaseUri();
+    
+    assertThat(modifiedUri.getScheme()).isEqualTo("https");
+    assertThat(modifiedUri.getUserInfo()).isNull();
+    assertThat(modifiedUri.getHost()).isEqualTo("localhost");
+    assertThat(modifiedUri.getPort()).isEqualTo(4444);
+    assertThat(modifiedUri.getPath()).isEqualTo("/wd/hub");
+    assertThat(modifiedUri.getQuery()).isEqualTo("debug=true");
+    assertThat(modifiedUri.getFragment()).isEqualTo("fragment");
+  }
 }