In the template C# file NuGetODataConfig.cs.pp, log exceptions to Trace (#53)

Author: joelverhagen

src/NuGet.Server/App_Start/NuGetODataConfig.cs

@@ -3,8 +3,10 @@
 
 using System.Net.Http;
 using System.Web.Http;
+using System.Web.Http.ExceptionHandling;
 using System.Web.Http.Routing;
 using NuGet.Server.DataServices;
+using NuGet.Server.Infrastructure;
 using NuGet.Server.V2;
 
 // The consuming project executes this logic with its own copy of this class. This is done with a .pp file that is
@@ -28,6 +30,11 @@ public static void Initialize(HttpConfiguration config, string controllerName)
         {
             NuGetV2WebApiEnabler.UseNuGetV2WebApiFeed(config, "NuGetDefault", "nuget", controllerName);
 
+            config.Services.Replace(typeof(IExceptionLogger), new TraceExceptionLogger());
+
+            // Trace.Listeners.Add(new TextWriterTraceListener(HostingEnvironment.MapPath("~/NuGet.Server.log")));
+            // Trace.AutoFlush = true;
+
             config.Routes.MapHttpRoute(
                 name: "NuGetDefault_ClearCache",
                 routeTemplate: "nuget/clear-cache",

src/NuGet.Server/App_Start/NuGetODataConfig.cs.pp

@@ -1,7 +1,9 @@
 using System.Net.Http;
 using System.Web.Http;
+using System.Web.Http.ExceptionHandling;
 using System.Web.Http.Routing;
 using NuGet.Server;
+using NuGet.Server.Infrastructure;
 using NuGet.Server.V2;
 
 [assembly: WebActivatorEx.PreApplicationStartMethod(typeof($rootnamespace$.App_Start.NuGetODataConfig), "Start")]
@@ -18,6 +20,11 @@
 
             NuGetV2WebApiEnabler.UseNuGetV2WebApiFeed(config, "NuGetDefault", "nuget", "PackagesOData");
 
+            config.Services.Replace(typeof(IExceptionLogger), new TraceExceptionLogger());
+
+            // Trace.Listeners.Add(new TextWriterTraceListener(HostingEnvironment.MapPath("~/NuGet.Server.log")));
+            // Trace.AutoFlush = true;
+
             config.Routes.MapHttpRoute(
                 name: "NuGetDefault_ClearCache",
                 routeTemplate: "nuget/clear-cache",

src/NuGet.Server/Infrastructure/TraceExceptionLogger.cs

@@ -0,0 +1,16 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using System.Diagnostics;
+using System.Web.Http.ExceptionHandling;
+
+namespace NuGet.Server.Infrastructure
+{
+    public class TraceExceptionLogger : ExceptionLogger
+    {
+        public override void Log(ExceptionLoggerContext context)
+        {
+            Trace.TraceError(context.ExceptionContext.Exception.ToString());
+        }
+    }
+}

src/NuGet.Server/NuGet.Server.csproj

@@ -109,6 +109,7 @@
     <Compile Include="Core\ServiceResolver.cs" />
     <Compile Include="Controllers\PackagesODataController.cs" />
     <Compile Include="Core\ServiceResolverExtensions.cs" />
+    <Compile Include="Infrastructure\TraceExceptionLogger.cs" />
     <Compile Include="Infrastructure\WebConfigSettingsProvider.cs" />
     <Compile Include="Core\Helpers.cs" />
     <Compile Include="Infrastructure\PackageAuthenticationService.cs" />