Merge pull request #66 from tintoy/fix/router-log-level

Enable opting out of built-in logger provider.
OmniSharp · Jan 31, 2018 · 6960413 · 6960413
2 parents 79abb1a + 05e2287
commit 6960413
Showing 1 changed file with 15 additions and 4 deletions.
diff --git a/src/Server/LanguageServer.cs b/src/Server/LanguageServer.cs
@@ -40,7 +40,16 @@ public LanguageServer(
             Stream input,
             Stream output,
             ILoggerFactory loggerFactory)
-            : this(input, output, new LspReciever(), new RequestProcessIdentifier(), loggerFactory, new Serializer())
+            : this(input, output, loggerFactory, addDefaultLoggingProvider: true)
+        {
+        }
+
+        public LanguageServer(
+            Stream input,
+            Stream output,
+            ILoggerFactory loggerFactory,
+            bool addDefaultLoggingProvider)
+            : this(input, output, new LspReciever(), new RequestProcessIdentifier(), loggerFactory, new Serializer(), addDefaultLoggingProvider)
         {
         }
 
@@ -50,11 +59,13 @@ internal LanguageServer(
             LspReciever reciever,
             IRequestProcessIdentifier requestProcessIdentifier,
             ILoggerFactory loggerFactory,
-            Serializer serializer)
+            Serializer serializer,
+            bool addDefaultLoggingProvider)
         {
             var outputHandler = new OutputHandler(output, serializer);
-            // TODO: This might not be the best
-            loggerFactory.AddProvider(new LanguageServerLoggerProvider(this));
+
+            if (addDefaultLoggingProvider)
+                loggerFactory.AddProvider(new LanguageServerLoggerProvider(this));
 
             _reciever = reciever;
             _loggerFactory = loggerFactory;