Description

I am trying to override /usr/bin/safaridriver as the safaridriver location, through a Grid Node. It seems ignored during my testing, and I cannot see a codepath that would allow it to be set through this env variable. To the contrary, I see it hardcoded in:

• selenium/java/src/org/openqa/selenium/safari/SafariDriverService.java

  Line 54 in 0fe5ed2

  private static final File SAFARI_DRIVER_EXECUTABLE = new File("/usr/bin/safaridriver");

• selenium/rust/src/safari.rs

  Line 113 in 0fe5ed2

  Ok(PathBuf::from("/usr/bin/safaridriver"))

If SE_SAFARIDRIVER is respected in python and ruby, why not in java or in selenium-manager?

Reproducible Code

# sh
SE_SAFARIDRIVER=/some/thing "$JAVA_HOME/bin/java" -jar selenium-server.jar node --log-level FINER

# py
safari_options = webdriver.SafariOptions()

driver = webdriver.Remote(
   command_executor="http://localhost:5555",
   options=safari_options
)
driver.get("https://www.google.com")


## The logs should show /usr/bin/safaridriver being used instead of /some/thing