fix: use environment to determine production logging

This fixes #18.

Author: Christoph Bühler

src/KubeOps/Operator/KubernetesOperator.cs

@@ -1,12 +1,14 @@
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using System.Reflection;
 using System.Threading.Tasks;
 using k8s;
 using KubeOps.Operator.Caching;
 using KubeOps.Operator.Client;
 using KubeOps.Operator.Commands;
 using KubeOps.Operator.DependencyInjection;
+using KubeOps.Operator.Logging;
 using KubeOps.Operator.Queue;
 using KubeOps.Operator.Serialization;
 using KubeOps.Operator.Watcher;
@@ -19,11 +21,6 @@
 using Newtonsoft.Json.Serialization;
 using YamlDotNet.Serialization;
 
-#if !DEBUG
-using System.Linq;
-using KubeOps.Operator.Logging;
-#endif
-
 namespace KubeOps.Operator
 {
     public sealed class KubernetesOperator
@@ -58,25 +55,29 @@ public Task<int> Run(string[] args)
             var app = new CommandLineApplication<RunOperator>();
 
             _builder.ConfigureLogging(
-                builder =>
+                (hostContext, logging) =>
                 {
-                    builder.ClearProviders();
-#if DEBUG
-                    builder.AddConsole(options => options.TimestampFormat = @"[HH:mm:ss] ");
-#else
-                    if (args.Contains(NoStructuredLogs))
+                    logging.ClearProviders();
+
+                    if (hostContext.HostingEnvironment.IsProduction())
                     {
-                        builder.AddConsole(options =>
+                        if (args.Contains(NoStructuredLogs))
                         {
-                            options.TimestampFormat = @"[dd.MM.yyyy - HH:mm:ss] ";
-                            options.DisableColors = true;
-                        });
+                            logging.AddConsole(options =>
+                            {
+                                options.TimestampFormat = @"[dd.MM.yyyy - HH:mm:ss] ";
+                                options.DisableColors = true;
+                            });
+                        }
+                        else
+                        {
+                            logging.AddStructuredConsole();
+                        }
                     }
                     else
                     {
-                        builder.AddStructuredConsole();
+                        logging.AddConsole(options => options.TimestampFormat = @"[HH:mm:ss] ");
                     }
-#endif
                 });
 
             var host = _builder.Build();
@@ -109,7 +110,7 @@ private void ConfigureRequiredServices() =>
                         {
                             ContractResolver = new NamingConvention(),
                             Converters = new List<JsonConverter>
-                                { new StringEnumConverter { NamingStrategy = new CamelCaseNamingStrategy() } },
+                                {new StringEnumConverter {NamingStrategy = new CamelCaseNamingStrategy()}},
                         });
                     services.AddTransient(
                         _ => new SerializerBuilder()
@@ -130,13 +131,13 @@ private void ConfigureRequiredServices() =>
                                 {
                                     ContractResolver = new NamingConvention(),
                                     Converters = new List<JsonConverter>
-                                        { new StringEnumConverter { NamingStrategy = new CamelCaseNamingStrategy() } }
+                                        {new StringEnumConverter {NamingStrategy = new CamelCaseNamingStrategy()}}
                                 },
                                 DeserializationSettings =
                                 {
                                     ContractResolver = new NamingConvention(),
                                     Converters = new List<JsonConverter>
-                                        { new StringEnumConverter { NamingStrategy = new CamelCaseNamingStrategy() } }
+                                        {new StringEnumConverter {NamingStrategy = new CamelCaseNamingStrategy()}}
                                 }
                             };
                         });