Fixing log Debug messages that should be log Trace messages; Commenting others

Author: StanleyGoldman

src/GitHub.Api/Authentication/LoginManager.cs

@@ -94,7 +94,7 @@ public async Task<LoginResultData> Login(
                 if (e is TwoFactorRequiredException)
                 {
                     result = LoginResultCodes.CodeRequired;
-                    logger.Debug("2FA TwoFactorAuthorizationException: {0} {1}", LoginResultCodes.CodeRequired, e.Message);
+                    logger.Trace("2FA TwoFactorAuthorizationException: {0} {1}", LoginResultCodes.CodeRequired, e.Message);
                 }
                 else
                 {
@@ -166,21 +166,21 @@ public async Task<LoginResultData> ContinueLogin(LoginResultData loginResultData
             }
             catch (TwoFactorAuthorizationException e)
             {
-                logger.Debug(e, "2FA TwoFactorAuthorizationException: {0} {1}", LoginResultCodes.CodeFailed, e.Message);
+                logger.Trace(e, "2FA TwoFactorAuthorizationException: {0} {1}", LoginResultCodes.CodeFailed, e.Message);
 
                 return new LoginResultData(LoginResultCodes.CodeFailed, Localization.Wrong2faCode, client, host, newAuth);
             }
             catch (ApiValidationException e)
             {
-                logger.Debug(e, "2FA ApiValidationException: {0}", e.Message);
+                logger.Trace(e, "2FA ApiValidationException: {0}", e.Message);
 
                 var message = e.ApiError.FirstErrorMessageSafe();
                 await keychain.Clear(host, false);
                 return new LoginResultData(LoginResultCodes.Failed, message, host);
             }
             catch (Exception e)
             {
-                logger.Debug(e, "Exception: {0}", e.Message);
+                logger.Trace(e, "Exception: {0}", e.Message);
 
                 await keychain.Clear(host, false);
                 return new LoginResultData(LoginResultCodes.Failed, e.Message, host);

src/GitHub.Api/Events/RepositoryWatcher.cs

@@ -71,7 +71,7 @@ public void Start()
         {
             if (disableNative)
             {
-                Logger.Debug("Native interface is disabled");
+                Logger.Trace("Native interface is disabled");
                 return;
             }
 

src/GitHub.Api/OutputProcessors/LockOutputProcessor.cs

@@ -29,7 +29,7 @@ public override void LineReceived(string line)
             {
                 return;
             }
-            Logger.Debug(line);
+            Logger.Trace(line);
             var path = proc.ReadUntil('\t').Trim();
             var user = proc.ReadUntilLast("ID:").Trim();
             proc.MoveToAfter("ID:");

src/UnityExtension/Assets/Editor/GitHub.Unity/Misc/Installer.cs

@@ -31,7 +31,7 @@ private static void MoveFrom(string path)
         {
             try
             {
-                logger.Debug("Installer move from '{0}'", path);
+                logger.Trace("Installer move from '{0}'", path);
                 // TODO: Create the necessary structure and perform the actual move of files into it from the given install path
             }
             catch (Exception e)

src/UnityExtension/Assets/Editor/GitHub.Unity/ScriptObjectSingleton.cs

@@ -115,7 +115,7 @@ private static NPath GetFilePath()
             var attr = typeof(T).GetCustomAttributes(true)
                                 .Select(t => t as LocationAttribute)
                                 .FirstOrDefault(t => t != null);
-            Logging.Instance.Debug("FilePath {0}", attr != null ? attr.filepath : null);
+            //Logging.Instance.Debug("FilePath {0}", attr != null ? attr.filepath : null);
 
             return attr != null ? attr.filepath.ToNPath() : null;
         }

src/UnityExtension/Assets/Editor/GitHub.Unity/UI/SettingsView.cs

@@ -703,7 +703,7 @@ private void ForceUnlockFile(object obj)
 
         private void Init()
         {
-            Logger.Debug("TODO: Init '{0}'", initDirectory);
+            //Logger.Debug("TODO: Init '{0}'", initDirectory);
         }
     }
 }