update: keep compatibility in mind

Author: kamal-kaur04

BrowserStackLocal/BrowserStackLocal/BrowserStackLocal.csproj

@@ -2,7 +2,7 @@
   <PropertyGroup>
     <RootNamespace>BrowserStackRootNamespace>
     <AssemblyName>BrowserStackLocalAssemblyName>
-    <TargetFrameworks>net48;netstandard2.0TargetFrameworks>
+    <TargetFrameworks>net20;netstandard2.0TargetFrameworks>
     <GenerateAssemblyInfo>falseGenerateAssemblyInfo>
     <Title>BrowserStackLocalTitle>
     <Product>BrowserStackLocalProduct>

BrowserStackLocal/BrowserStackLocal/BrowserStackTunnel.cs

@@ -7,17 +7,18 @@
 using System.Security.AccessControl;
 using System.Security.Principal;
 using Newtonsoft.Json.Linq;
-using System.Runtime.InteropServices;
 
 namespace BrowserStack
 {
   public enum LocalState { Idle, Connecting, Connected, Error, Disconnected };
 
   public class BrowserStackTunnel : IDisposable
   {
-    static string binaryName = GetBinaryName();
+    static readonly string uname = Util.GetUName();
+    static readonly string binaryName = GetBinaryName();
     static readonly string downloadURL = "https://www.browserstack.com/local-testing/downloads/binaries/" + binaryName;
-    static readonly string homepath = IsDarwin() || IsLinux() ?
+
+    static readonly string homepath = !IsWindows() ?
                                         Environment.GetFolderPath(Environment.SpecialFolder.Personal) :
                                         Environment.ExpandEnvironmentVariables("%HOMEDRIVE%%HOMEPATH%");
     public static readonly string[] basePaths = new string[] {
@@ -36,9 +37,9 @@ public class BrowserStackTunnel : IDisposable
 
     Process process = null;
 
-    static bool IsDarwin()
+    static bool IsDarwin(string osName)
     {
-      if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
+      if(osName.Contains("darwin"))
       {
         return true;
       }
@@ -48,16 +49,16 @@ static bool IsDarwin()
 
     static bool IsWindows()
     {
-      if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
+      if (Environment.OSVersion.VersionString?.ToLower().Contains("windows") ?? false)
       {
         return true;
       }
       return false;
     }
 
-    static bool IsLinux()
+    static bool IsLinux(string osName)
     {
-      if (RuntimeInformation.IsOSPlatform(OSPlatform.Linux))
+      if(osName.Contains("linux"))
       {
         return true;
       }
@@ -69,7 +70,7 @@ static bool IsAlpine()
       try
       {
         string[] output = Util.RunShellCommand("grep -w 'NAME' /etc/os-release");
-        if (string.IsNullOrEmpty(output[0]) && output[0].Contains("Alpine"))
+        if (output[0]?.ToLower()?.Contains("alpine") ?? false)
         {
           return true;
         }
@@ -83,17 +84,17 @@ static bool IsAlpine()
 
     static string GetBinaryName()
     {
-      if (IsDarwin())
+      if (IsWindows())
       {
-        return "BrowserStackLocal-darwin-x64";
+        return "BrowserStackLocal.exe";
       }
-      else if (IsWindows())
+      else if (IsDarwin(uname))
       {
-        return "BrowserStackLocal.exe";
+        return "BrowserStackLocal-darwin-x64";
       }
-      else if (IsLinux())
+      else if (IsLinux(uname))
       {
-        if (Environment.Is64BitOperatingSystem)
+        if (Util.Is64BitOS())
         {
           if (IsAlpine())
           {
@@ -148,7 +149,7 @@ public virtual void fallbackPaths()
 
     public void modifyBinaryPermission()
     {
-      if (IsDarwin() || IsLinux())
+      if (!IsWindows())
        {
         try
         {

BrowserStackLocal/BrowserStackLocal/Util.cs

@@ -1,11 +1,21 @@
+using System;
 using System.Diagnostics;
+using System.Runtime.InteropServices;
 
 namespace BrowserStack
 {
     public class Util {
 
+        [DllImport("kernel32.dll", SetLastError = true, CallingConvention = CallingConvention.Winapi)]
+        [return: MarshalAs(UnmanagedType.Bool)]
+        private static extern bool IsWow64Process(
+            [In] IntPtr hProcess,
+            [Out] out bool wow64Process
+        );
+
         // Only Unix Support
-        public static string[] RunShellCommand(string command) {
+        public static string[] RunShellCommand(string command)
+        {
             ProcessStartInfo psi = new ProcessStartInfo("bash", $"-c \"{command}\"") {
                 RedirectStandardOutput = true,
                 RedirectStandardError = true,
@@ -20,7 +30,45 @@ public static string[] RunShellCommand(string command) {
             process.WaitForExit();
             return new string[]{output, error};
         }
-        
+
+        public static string GetUName()
+        {
+            string osName = "";
+            try
+            {
+                string[] output = RunShellCommand("uname");
+                osName = output[0]?.ToLower();
+            }
+            catch (System.Exception) {}
+            return osName;
+        }
+
+        public static bool Is64BitOS()
+        {
+            bool is64BitProcess = IntPtr.Size == 8;
+            return is64BitProcess || InternalCheckIsWow64();
+        }
+
+        public static bool InternalCheckIsWow64()
+        {
+            if ((Environment.OSVersion.Version.Major == 5 && Environment.OSVersion.Version.Minor >= 1) ||
+                Environment.OSVersion.Version.Major >= 6)
+            {
+                using (Process p = Process.GetCurrentProcess())
+                {
+                    bool retVal;
+                    if (!IsWow64Process(p.Handle, out retVal))
+                    {
+                        return false;
+                    }
+                    return retVal;
+                }
+            }
+            else
+            {
+                return false;
+            }
+        }
     }
 }