merge latest

Author: aguspe

.github/workflows/ci-python.yml

@@ -20,10 +20,10 @@ jobs:
     steps:
       - name: Checkout source tree
         uses: actions/checkout@v4
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
@@ -40,10 +40,10 @@ jobs:
     steps:
       - name: Checkout source tree
         uses: actions/checkout@v4
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
@@ -62,10 +62,10 @@ jobs:
     steps:
       - name: Checkout source tree
         uses: actions/checkout@v4
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v4
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip

.github/workflows/nightly.yml

@@ -172,7 +172,7 @@ jobs:
     with:
       name: Nightly JavaScript Release
       cache-key: javascript-nightly
-      node-version: '18.x'
+      node-version: '22.x'
       run: |
         sed -i 's|https://registry.npmjs.org/|https://npm.pkg.github.com|g' javascript/node/selenium-webdriver/package.json
         sed -i 's|"name": "selenium-webdriver"|"name": "@seleniumhq/selenium-webdriver"|g' javascript/node/selenium-webdriver/package.json

.github/workflows/update-documentation.yml

@@ -107,10 +107,10 @@ jobs:
         run: |
           git config --local user.email "[email protected]"
           git config --local user.name "Selenium CI Bot"
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v5
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip

MODULE.bazel

@@ -111,11 +111,11 @@ use_repo(oci, "chrome_standalone", "firefox_standalone", "java_image_base")
 python = use_extension("@rules_python//python/extensions:python.bzl", "python")
 python.toolchain(
     is_default = True,
-    python_version = "3.8",
+    python_version = "3.9",
 )
-python.toolchain(python_version = "3.9")
 python.toolchain(python_version = "3.10")
 python.toolchain(python_version = "3.11")
+python.toolchain(python_version = "3.12")
 use_repo(python, "pythons_hub")
 
 pip = use_extension("@rules_python//python/extensions:pip.bzl", "pip")
@@ -127,10 +127,10 @@ pip = use_extension("@rules_python//python/extensions:pip.bzl", "pip")
         requirements_lock = "//py:requirements_lock.txt",
     )
     for version in [
-        "3.8",
         "3.9",
         "3.10",
         "3.11",
+        "3.12",
     ]
 ]
 
@@ -216,7 +216,7 @@ maven.install(
         "org.junit.platform:junit-platform-reporting",
         "org.junit.platform:junit-platform-commons",
         "org.junit.platform:junit-platform-engine",
-        "org.mockito:mockito-core:5.14.2",
+        "org.mockito:mockito-core:5.15.2",
         "org.redisson:redisson:3.41.0",
         "org.slf4j:slf4j-api:2.0.16",
         "org.slf4j:slf4j-jdk14:2.0.16",

dotnet/src/webdriver/Safari/SafariDriverService.cs

@@ -87,56 +87,6 @@ protected override bool HasShutdown
             get { return false; }
         }
 
-        /// <summary>
-        /// Gets a value indicating whether the service is responding to HTTP requests.
-        /// </summary>
-        protected override bool IsInitialized
-        {
-            get
-            {
-                bool isInitialized = false;
-
-                Uri serviceHealthUri = new Uri(this.ServiceUrl, new Uri("/session/FakeSessionIdForPollingPurposes", UriKind.Relative));
-
-                // Since Firefox driver won't implement the /session end point (because
-                // the W3C spec working group stupidly decided that it isn't necessary),
-                // we'll attempt to poll for a different URL which has no side effects.
-                // We've chosen to poll on the "quit" URL, passing in a nonexistent
-                // session id.
-                using (var httpClient = new HttpClient())
-                {
-                    httpClient.DefaultRequestHeaders.ConnectionClose = true;
-                    httpClient.Timeout = TimeSpan.FromSeconds(5);
-
-                    using (var httpRequest = new HttpRequestMessage(HttpMethod.Delete, serviceHealthUri))
-                    {
-                        try
-                        {
-                            using (var httpResponse = Task.Run(async () => await httpClient.SendAsync(httpRequest)).GetAwaiter().GetResult())
-                            {
-                                isInitialized = (httpResponse.StatusCode == HttpStatusCode.OK
-                                        || httpResponse.StatusCode == HttpStatusCode.InternalServerError
-                                        || httpResponse.StatusCode == HttpStatusCode.NotFound)
-                                    && httpResponse.Content.Headers.ContentType.MediaType.StartsWith("application/json", StringComparison.OrdinalIgnoreCase);
-                            }
-                        }
-
-                        // Checking the response from deleting a nonexistent session. Note that we are simply
-                        // checking that the HTTP status returned is a 200 status, and that the resposne has
-                        // the correct Content-Type header. A more sophisticated check would parse the JSON
-                        // response and validate its values. At the moment we do not do this more sophisticated
-                        // check.
-                        catch (Exception ex) when (ex is HttpRequestException || ex is TaskCanceledException)
-                        {
-                            // Do nothing. The exception is expected, meaning driver service is not initialized.
-                        }
-                    }
-                }
-
-                return isInitialized;
-            }
-        }
-
         /// <summary>
         /// Creates a default instance of the SafariDriverService.
         /// </summary>

dotnet/src/webdriver/assets/README.md

@@ -6,13 +6,11 @@ Selenium is a set of different software tools each with a different approach to
 using OpenQA.Selenium.Chrome;
 using OpenQA.Selenium;
 
-var driver = new ChromeDriver();
+using var driver = new ChromeDriver();
 
 driver.Url = "https://www.google.com";
 driver.FindElement(By.Name("q")).SendKeys("webdriver" + Keys.Return);
 Console.WriteLine(driver.Title);
-
-driver.Quit();
 ```
 
 # Contributing

java/maven_install.json

@@ -1,7 +1,7 @@
 {
   "__AUTOGENERATED_FILE_DO_NOT_MODIFY_THIS_FILE_MANUALLY": "THERE_IS_NO_DATA_ONLY_ZUUL",
-  "__INPUT_ARTIFACTS_HASH": 1132645732,
-  "__RESOLVED_ARTIFACTS_HASH": 1947623231,
+  "__INPUT_ARTIFACTS_HASH": 1220324005,
+  "__RESOLVED_ARTIFACTS_HASH": 1655871984,
   "artifacts": {
     "com.beust:jcommander": {
       "shasums": {
@@ -459,10 +459,10 @@
     },
     "net.bytebuddy:byte-buddy-agent": {
       "shasums": {
-        "jar": "1d76defd159a564b9cb7a968d0dea27367b8b70ebde75a968e7ef1921bc75ee4",
-        "sources": "8a7e537a5c2a932a6d73dcec9aa8d5388d33ab3aa91ea410aeea5228dcfe9745"
+        "jar": "316d2c0795c2a4d4c4756f2e6f9349837c7430ac34e0477ead874d05f5cc19e5",
+        "sources": "a73d2869afd79b63396a0a07ac037bbfa184eb7306916343c030feb5335ac940"
       },
-      "version": "1.15.4"
+      "version": "1.15.11"
     },
     "net.sf.saxon:Saxon-HE": {
       "shasums": {
@@ -676,10 +676,10 @@
     },
     "org.mockito:mockito-core": {
       "shasums": {
-        "jar": "2296141c1e1f2e1ae35c08d36a9ab4563ecd66e03533fe82630a764e7aa49182",
-        "sources": "32f318184ab3795885743f23d8be0da7fe856e3e360b518083f8b366a44d2b33"
+        "jar": "bf48b7372d9491d5ec8aebb4cdd187d15663931599c0fbe7410166ce0e1e58ff",
+        "sources": "bc0ee4bc1d8d43a9bd5e5b49dc479566c28db0740284524140aa264e57e8e27e"
       },
-      "version": "5.14.2"
+      "version": "5.15.2"
     },
     "org.objenesis:objenesis": {
       "shasums": {

java/src/org/openqa/selenium/Cookie.java

@@ -24,18 +24,21 @@
 import java.util.Objects;
 import java.util.TimeZone;
 import java.util.TreeMap;
+import org.jspecify.annotations.NullMarked;
+import org.jspecify.annotations.Nullable;
 
+@NullMarked
 public class Cookie implements Serializable {
   private static final long serialVersionUID = 4115876353625612383L;
 
   private final String name;
   private final String value;
   private final String path;
-  private final String domain;
-  private final Date expiry;
+  private final @Nullable String domain;
+  private final @Nullable Date expiry;
   private final boolean isSecure;
   private final boolean isHttpOnly;
-  private final String sameSite;
+  private final @Nullable String sameSite;
 
   /**
    * Creates an insecure non-httpOnly cookie with no domain specified.
@@ -47,7 +50,7 @@ public class Cookie implements Serializable {
    * @param expiry The cookie's expiration date; may be null.
    * @see #Cookie(String, String, String, String, Date)
    */
-  public Cookie(String name, String value, String path, Date expiry) {
+  public Cookie(String name, String value, @Nullable String path, @Nullable Date expiry) {
     this(name, value, null, path, expiry);
   }
 
@@ -62,7 +65,12 @@ public Cookie(String name, String value, String path, Date expiry) {
    * @param expiry The cookie's expiration date; may be null.
    * @see #Cookie(String, String, String, String, Date, boolean)
    */
-  public Cookie(String name, String value, String domain, String path, Date expiry) {
+  public Cookie(
+      String name,
+      String value,
+      @Nullable String domain,
+      @Nullable String path,
+      @Nullable Date expiry) {
     this(name, value, domain, path, expiry, false);
   }
 
@@ -78,7 +86,12 @@ public Cookie(String name, String value, String domain, String path, Date expiry
    * @param isSecure Whether this cookie requires a secure connection.
    */
   public Cookie(
-      String name, String value, String domain, String path, Date expiry, boolean isSecure) {
+      String name,
+      String value,
+      @Nullable String domain,
+      @Nullable String path,
+      @Nullable Date expiry,
+      boolean isSecure) {
     this(name, value, domain, path, expiry, isSecure, false);
   }
 
@@ -97,9 +110,9 @@ public Cookie(
   public Cookie(
       String name,
       String value,
-      String domain,
-      String path,
-      Date expiry,
+      @Nullable String domain,
+      @Nullable String path,
+      @Nullable Date expiry,
       boolean isSecure,
       boolean isHttpOnly) {
     this(name, value, domain, path, expiry, isSecure, isHttpOnly, null);
@@ -121,12 +134,12 @@ public Cookie(
   public Cookie(
       String name,
       String value,
-      String domain,
-      String path,
-      Date expiry,
+      @Nullable String domain,
+      @Nullable String path,
+      @Nullable Date expiry,
       boolean isSecure,
       boolean isHttpOnly,
-      String sameSite) {
+      @Nullable String sameSite) {
     this.name = name;
     this.value = value;
     this.path = path == null || path.isEmpty() ? "/" : path;
@@ -174,7 +187,7 @@ public String getValue() {
     return value;
   }
 
-  public String getDomain() {
+  public @Nullable String getDomain() {
     return domain;
   }
 
@@ -190,15 +203,15 @@ public boolean isHttpOnly() {
     return isHttpOnly;
   }
 
-  public Date getExpiry() {
+  public @Nullable Date getExpiry() {
     return expiry == null ? null : new Date(expiry.getTime());
   }
 
-  public String getSameSite() {
+  public @Nullable String getSameSite() {
     return sameSite;
   }
 
-  private static String stripPort(String domain) {
+  private static @Nullable String stripPort(@Nullable String domain) {
     return (domain == null) ? null : domain.split(":")[0];
   }
 
@@ -270,7 +283,7 @@ public String toString() {
 
   /** Two cookies are equal if the name and value match */
   @Override
-  public boolean equals(Object o) {
+  public boolean equals(@Nullable Object o) {
     if (this == o) {
       return true;
     }
@@ -295,29 +308,29 @@ public static class Builder {
 
     private final String name;
     private final String value;
-    private String path;
-    private String domain;
-    private Date expiry;
+    private @Nullable String path;
+    private @Nullable String domain;
+    private @Nullable Date expiry;
     private boolean secure;
     private boolean httpOnly;
-    private String sameSite;
+    private @Nullable String sameSite;
 
     public Builder(String name, String value) {
       this.name = name;
       this.value = value;
     }
 
-    public Builder domain(String host) {
+    public Builder domain(@Nullable String host) {
       this.domain = stripPort(host);
       return this;
     }
 
-    public Builder path(String path) {
+    public Builder path(@Nullable String path) {
       this.path = path;
       return this;
     }
 
-    public Builder expiresOn(Date expiry) {
+    public Builder expiresOn(@Nullable Date expiry) {
       this.expiry = expiry == null ? null : new Date(expiry.getTime());
       return this;
     }
@@ -332,7 +345,7 @@ public Builder isHttpOnly(boolean httpOnly) {
       return this;
     }
 
-    public Builder sameSite(String sameSite) {
+    public Builder sameSite(@Nullable String sameSite) {
       this.sameSite = sameSite;
       return this;
     }