report language version to plausible

Author: titusfortner

dotnet/src/webdriver/SeleniumManager.cs

@@ -178,6 +178,13 @@ public static Dictionary<string, string> BinaryPaths(string arguments)
     {
         StringBuilder argsBuilder = new StringBuilder(arguments);
         argsBuilder.Append(" --language-binding csharp");
+#if NET8_0_OR_GREATER
+        argsBuilder.Append($" --language-version net8.0/{Environment.Version.Major}");
+#elif NETSTANDARD2_0
+        argsBuilder.Append($" --language-version netstandard2.0/{Environment.Version.Major}");
+#elif NET462
+        argsBuilder.Append(" --language-version net462");
+#endif
         argsBuilder.Append(" --output json");
         if (_logger.IsEnabled(LogEventLevel.Debug))
         {

java/src/org/openqa/selenium/manager/SeleniumManager.java

@@ -261,10 +261,12 @@ private void saveToFileSafely(InputStream inputStream, Path target) throws IOExc
    * @return the locations of the assets from Selenium Manager execution
    */
   public Result getBinaryPaths(List<String> arguments) {
-    List<String> args = new ArrayList<>(arguments.size() + 5);
+    List<String> args = new ArrayList<>(arguments.size() + 7);
     args.addAll(arguments);
     args.add("--language-binding");
     args.add("java");
+    args.add("--language-version");
+    args.add(Runtime.version().feature() + "");
     args.add("--output");
     args.add("json");
 

javascript/selenium-webdriver/common/driverFinder.js

@@ -44,7 +44,16 @@ function getBinaryPaths(capabilities) {
 }
 
 function getArgs(options) {
-  let args = ['--browser', options.getBrowserName(), '--language-binding', 'javascript', '--output', 'json']
+  let args = [
+    '--browser',
+    options.getBrowserName(),
+    '--language-binding',
+    'javascript',
+    '--language-version',
+    process.versions.node,
+    '--output',
+    'json',
+  ]
 
   if (options.getBrowserVersion() && options.getBrowserVersion() !== '') {
     args.push('--browser-version', options.getBrowserVersion())

py/selenium/webdriver/common/selenium_manager.py

@@ -49,6 +49,8 @@ def binary_paths(self, args: list) -> dict:
             args.append("--debug")
         args.append("--language-binding")
         args.append("python")
+        args.append("--language-version")
+        args.append(f"{sys.version_info.major}.{sys.version_info.minor}")
         args.append("--output")
         args.append("json")
 

rb/lib/selenium/webdriver/common/selenium_manager.rb

@@ -38,6 +38,7 @@ def bin_path
         # @return [Hash] paths to the requested assets.
         def binary_paths(*arguments)
           arguments += %w[--language-binding ruby]
+          arguments += ['--language-version', RUBY_VERSION]
           arguments += %w[--output json]
           arguments << '--debug' if WebDriver.logger.debug?
 

rust/src/config.rs

@@ -64,6 +64,7 @@ pub struct ManagerConfig {
     pub force_browser_download: bool,
     pub avoid_browser_download: bool,
     pub language_binding: String,
+    pub language_binding_version: String,
     pub selenium_version: String,
     pub avoid_stats: bool,
     pub skip_driver_in_path: bool,
@@ -131,6 +132,7 @@ impl ManagerConfig {
             force_browser_download: BooleanKey("force-browser-download", false).get_value(),
             avoid_browser_download: BooleanKey("avoid-browser-download", false).get_value(),
             language_binding: StringKey(vec!["language-binding"], "").get_value(),
+            language_binding_version: StringKey(vec!["language-version"], "").get_value(),
             selenium_version: StringKey(vec!["selenium-version"], "").get_value(),
             avoid_stats: BooleanKey("avoid-stats", false).get_value(),
             skip_driver_in_path: BooleanKey("skip-driver-in-path", false).get_value(),

rust/src/lib.rs

@@ -908,6 +908,7 @@ pub trait SeleniumManager {
                     .unwrap_or(ARCH_OTHER)
                     .to_ascii_lowercase(),
                 lang: self.get_language_binding().to_ascii_lowercase(),
+                lang_version: self.get_language_binding_version().to_ascii_lowercase(),
                 selenium_version: self.get_selenium_version().to_ascii_lowercase(),
             };
             let http_client = self.get_http_client().to_owned();
@@ -1605,6 +1606,16 @@ pub trait SeleniumManager {
         }
     }
 
+    fn get_language_binding_version(&self) -> &str {
+        self.get_config().language_binding_version.as_str()
+    }
+
+    fn set_language_binding_version(&mut self, language_binding_version: String) {
+        if !language_binding_version.is_empty() {
+            self.get_config_mut().language_binding_version = language_binding_version;
+        }
+    }
+
     fn get_selenium_version(&self) -> &str {
         self.get_config().selenium_version.as_str()
     }

rust/src/main.rs

@@ -145,6 +145,10 @@ struct Cli {
     #[clap(long)]
     language_binding: Option<String>,
 
+    /// Version of the language binding invoking Selenium Manager (e.g., 4.18.1, 3.12, net8.0/8)
+    #[clap(long)]
+    language_version: Option<String>,
+
     /// Avoid sends usage statistics to plausible.io
     #[clap(long)]
     avoid_stats: bool,
@@ -223,6 +227,7 @@ fn main() {
     selenium_manager.set_cache_path(cache_path.clone());
     selenium_manager.set_offline(cli.offline);
     selenium_manager.set_language_binding(cli.language_binding.unwrap_or_default());
+    selenium_manager.set_language_binding_version(cli.language_version.unwrap_or_default());
     let sm_version = clap::crate_version!();
     let selenium_version = sm_version.strip_prefix(SM_BETA_LABEL).unwrap_or(sm_version);
     selenium_manager.set_selenium_version(selenium_version.to_string());

rust/src/metadata.rs

@@ -32,6 +32,7 @@ pub struct Stats {
     pub os: String,
     pub arch: String,
     pub lang: String,
+    pub lang_version: String,
     pub selenium_version: String,
     pub stats_ttl: u64,
 }
@@ -147,6 +148,7 @@ pub fn is_stats_in_metadata(stats_metadata: &[Stats], props: &Props) -> bool {
                 && p.os.eq(&props.os)
                 && p.arch.eq(&props.arch)
                 && p.lang.eq(&props.lang)
+                && p.lang_version.eq(&props.lang_version)
                 && p.selenium_version.eq(&props.selenium_version)
         })
         .collect();
@@ -188,6 +190,7 @@ pub fn create_stats_metadata(props: &Props, stats_ttl: u64) -> Stats {
         os: props.os.to_string(),
         arch: props.arch.to_string(),
         lang: props.lang.to_string(),
+        lang_version: props.lang_version.to_string(),
         selenium_version: props.selenium_version.to_string(),
         stats_ttl: now_unix_timestamp() + stats_ttl,
     }

rust/src/stats.rs

@@ -46,6 +46,7 @@ pub struct Props {
     pub os: String,
     pub arch: String,
     pub lang: String,
+    pub lang_version: String,
     pub selenium_version: String,
 }