remove logging

Author: vedharish

BrowserStackLocal/BrowserStackLocal/BrowserStackTunnel.cs

@@ -1,14 +1,11 @@
 using System;
 using System.IO;
-using System.Collections.Generic;
 using System.Diagnostics;
 using System.Text;
-using System.Text.RegularExpressions;
 using System.Net;
 
 using System.Security.AccessControl;
 using System.Security.Principal;
-using System.Threading;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 
@@ -78,14 +75,11 @@ public void downloadBinary()
 
       using (var client = new WebClient())
       {
-        Local.logger.Info("Downloading BrowserStackLocal..");
         client.DownloadFile(downloadURL, this.binaryAbsolute);
-        Local.logger.Info("Binary Downloaded.");
       }
 
       if (!File.Exists(binaryAbsolute))
       {
-        Local.logger.Error("Error accessing downloaded zip. Please check file permissions.");
         throw new Exception("Error accessing file " + binaryAbsolute);
       }
 
@@ -108,7 +102,6 @@ public virtual void Run(string accessKey, string folder, string logFilePath, str
       }
       if (!File.Exists(binaryAbsolute))
       {
-        Local.logger.Warn("BrowserStackLocal binary was not found at " + binaryAbsolute);
         downloadBinary();
       }
 
@@ -121,8 +114,6 @@ public virtual void Run(string accessKey, string folder, string logFilePath, str
       {
         File.WriteAllText(logFilePath, string.Empty);
       }
-      Local.logger.Info("BrowserStackLocal binary is located at " + binaryAbsolute);
-      Local.logger.Info("Starting Binary with arguments " + arguments.Replace(accessKey, "<access_key>"));
       RunProcess(arguments, processType); 
     }
 

BrowserStackLocal/BrowserStackLocal/Local.cs

@@ -1,9 +1,3 @@
-using log4net;
-using log4net.Appender;
-using log4net.Core;
-using log4net.Filter;
-using log4net.Layout;
-using log4net.Repository.Hierarchy;
 using System;
 using System.Collections.Generic;
 using System.IO;
@@ -13,15 +7,12 @@ namespace BrowserStack
 {
   public class Local
   {
-    private Hierarchy hierarchy;
     private string folder = "";
     private string accessKey = "";
     private string customLogPath = "";
     private string argumentString = "";
     private string customBinaryPath = "";
-    private PatternLayout patternLayout;
     protected BrowserStackTunnel tunnel = null;
-    public static ILog logger = LogManager.GetLogger("Local");
     private static KeyValuePair<string, string> emptyStringPair = new KeyValuePair<string, string>();
 
     private static List<KeyValuePair<string, string>> valueCommands = new List<KeyValuePair<string, string>>() {
@@ -102,34 +93,9 @@ private void addArgs(string key, string value)
         }
       }
     }
-    private void setupLogging()
-    {
-      hierarchy = (Hierarchy)LogManager.GetRepository();
-
-      patternLayout = new PatternLayout();
-      patternLayout.ConversionPattern = "%date [%thread] %-5level %logger - %message%newline";
-      patternLayout.ActivateOptions();
-
-      ConsoleAppender consoleAppender = new ConsoleAppender();
-      consoleAppender.Threshold = Level.Info;
-      consoleAppender.Layout = patternLayout;
-      consoleAppender.ActivateOptions();
-
-      LoggerMatchFilter loggerMatchFilter = new LoggerMatchFilter();
-      loggerMatchFilter.LoggerToMatch = "Local";
-      loggerMatchFilter.AcceptOnMatch = true;
-      consoleAppender.AddFilter(loggerMatchFilter);
-      consoleAppender.AddFilter(new DenyAllFilter());
-
-      hierarchy.Root.AddAppender(consoleAppender);
-
-      hierarchy.Root.Level = Level.All;
-      hierarchy.Configured = true;
-    }
 
     public Local()
     {
-      setupLogging();
       tunnel = new BrowserStackTunnel();
     }
     public void start(List<KeyValuePair<string, string>> options)
@@ -166,7 +132,6 @@ public void start(List<KeyValuePair<string, string>> options)
           tunnel.Run(accessKey, folder, customLogPath, "start");
         } catch (Exception)
         {
-          logger.Warn("Running Local failed. Falling back to backup path.");
           except = true;
         }
         if (except)