[java] Improving logging to understand why "Value must be set" is rai…

…sed.
SeleniumHQ · May 2, 2024 · a0a3914 · a0a3914
1 parent 86fd063
commit a0a3914
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/java/src/org/openqa/selenium/grid/node/config/DriverServiceSessionFactory.java b/java/src/org/openqa/selenium/grid/node/config/DriverServiceSessionFactory.java
@@ -33,6 +33,7 @@
 import java.util.Set;
 import java.util.function.Function;
 import java.util.function.Predicate;
+import java.util.logging.Level;
 import java.util.logging.Logger;
 import java.util.stream.Stream;
 import org.openqa.selenium.Capabilities;
@@ -329,10 +330,11 @@ private Capabilities setBrowserBinary(Capabilities options, String browserPath)
               .setCapability(vendorOptionsCapability, vendorOptions)
               .setCapability("browserVersion", null);
         } catch (Exception e) {
-          LOG.warning(
+          LOG.log(
+              Level.WARNING,
               String.format(
-                  "Exception while setting the browser binary path. %s: %s",
-                  options, e.getMessage()));
+                  "Exception while setting the browser binary path. Options: %s", options),
+              e);
         }
       }
     }