Exclude some messages from the binlog

Author: jjonescz

src/Compilers/Core/MSBuildTask/ManagedCompiler.cs

@@ -540,7 +540,9 @@ internal int ExecuteTool(string pathToTool, string responseFileCommands, string
 
                 _sharedCompileCts = new CancellationTokenSource();
                 logger.Log($"CommandLine = '{commandLineCommands}'");
-                logger.Log($"BuildResponseFile = '{responseFileCommands}'");
+                logger.Log($"BuildResponseFile = '{responseFileCommands}'",
+                    // Command line args can be long and are already included in the binlog.
+                    excludeFromBinlog: true);
 
                 var clientDirectory = Path.GetDirectoryName(PathToBuiltInTool);
                 if (clientDirectory is null || tempDirectory is null)
@@ -813,7 +815,7 @@ private void LogCompilationMessage(ICompilerServerLogger logger, string requestI
             var message = $"CompilerServer: {category} - {diagnostic} - {requestId}";
             if (kind == CompilationKind.FatalError)
             {
-                logger.LogError(message);
+                logger.LogError(message, excludeFromBinlog: true);
                 Log.LogError(message);
             }
             else

src/Compilers/Core/MSBuildTask/TaskCompilerServerLogger.cs

@@ -20,9 +20,9 @@ internal sealed class TaskCompilerServerLogger(
 
     public bool IsLogging => true;
 
-    public void Log(string message)
+    public void Log(string message, bool excludeFromBinlog = false)
     {
         _inner.Log(message);
-        _taskLogger.LogMessage(message);
+        if (!excludeFromBinlog) _taskLogger.LogMessage(message);
     }
 }

src/Compilers/Shared/CompilerServerLogger.cs

@@ -24,7 +24,7 @@ namespace Microsoft.CodeAnalysis.CommandLine
     internal interface ICompilerServerLogger
     {
         bool IsLogging { get; }
-        void Log(string message);
+        void Log(string message, bool excludeFromBinlog = false);
     }
 
     internal static class CompilerServerLoggerExtensions
@@ -37,11 +37,11 @@ internal static void Log(this ICompilerServerLogger logger, string format, param
             }
         }
 
-        internal static void LogError(this ICompilerServerLogger logger, string message)
+        internal static void LogError(this ICompilerServerLogger logger, string message, bool excludeFromBinlog = false)
         {
             if (logger.IsLogging)
             {
-                logger.Log($"Error: {message}");
+                logger.Log($"Error: {message}", excludeFromBinlog: excludeFromBinlog);
             }
         }
 
@@ -146,7 +146,7 @@ public void Dispose()
             _loggingStream = null;
         }
 
-        public void Log(string message)
+        public void Log(string message, bool excludeFromBinlog = false)
         {
             if (_loggingStream is object)
             {
@@ -174,7 +174,7 @@ private EmptyCompilerServerLogger()
         {
         }
 
-        public void Log(string message)
+        public void Log(string message, bool excludeFromBinlog = false)
         {
         }
     }