NLog (#426)

Author: lampenlampen

Files/App.xaml.cs

@@ -25,6 +25,7 @@
 using Files.CommandLine;
 using Files.View_Models;
 using Files.Controls;
+using NLog;
 
 namespace Files
 {
@@ -56,23 +57,40 @@ public static IShellPage CurrentInstance
         public static ObservableCollection<WSLDistroItem> linuxDistroItems = new ObservableCollection<WSLDistroItem>();
         public static SettingsViewModel AppSettings { get; set; }
 
+        private static readonly Logger Logger = LogManager.GetCurrentClassLogger();
+
         public App()
         {
 	        this.InitializeComponent();
             this.Suspending += OnSuspending;
+
+            // Initialize NLog
+            Windows.Storage.StorageFolder storageFolder = Windows.Storage.ApplicationData.Current.LocalFolder;
+            NLog.LogManager.Configuration.Variables["LogPath"] = storageFolder.Path;
+
+            RegisterUncaughtExceptionLogger();
+
             consentDialog = new Dialogs.ConsentDialog();
             propertiesDialog = new Dialogs.PropertiesDialog();
             layoutDialog = new Dialogs.LayoutDialog();
             addItemDialog = new Dialogs.AddItemDialog();
             exceptionDialog = new Dialogs.ExceptionDialog();
-            //this.UnhandledException += App_UnhandledException;
+            // this.UnhandledException += App_UnhandledException;
             Clipboard.ContentChanged += Clipboard_ContentChanged;
             Clipboard_ContentChanged(null, null);
             AppCenter.Start("682666d1-51d3-4e4a-93d0-d028d43baaa0", typeof(Analytics), typeof(Crashes));
 
             AppSettings = new SettingsViewModel();
         }
 
+        private void RegisterUncaughtExceptionLogger()
+        {
+	        UnhandledException += (sender, args) =>
+	        {
+		        Logger.Error(args.Exception, args.Message);
+	        };
+        }
+
         private void CoreWindow_PointerPressed(CoreWindow sender, PointerEventArgs args)
         {
             if (args.CurrentPoint.Properties.IsXButton1Pressed)
@@ -172,7 +190,9 @@ public static IReadOnlyList<ContentDialog> FindDisplayedContentDialogs<T>()
         /// <param name="e">Details about the launch request and process.</param>
         protected override void OnLaunched(LaunchActivatedEventArgs e)
         {
-            bool canEnablePrelaunch = Windows.Foundation.Metadata.ApiInformation.IsMethodPresent("Windows.ApplicationModel.Core.CoreApplication", "EnablePrelaunch");
+            Logger.Info("App launched");
+
+	        bool canEnablePrelaunch = Windows.Foundation.Metadata.ApiInformation.IsMethodPresent("Windows.ApplicationModel.Core.CoreApplication", "EnablePrelaunch");
 
             Frame rootFrame = Window.Current.Content as Frame;
 
@@ -220,6 +240,8 @@ protected override void OnLaunched(LaunchActivatedEventArgs e)
 
         protected override void OnActivated(IActivatedEventArgs args)
         {
+            Logger.Info("App activated");
+
             // Window management
             Frame rootFrame = Window.Current.Content as Frame;
             if (rootFrame == null)

Files/Files.csproj

@@ -278,6 +278,9 @@
     <None Include="Assets\terminal\terminal.json">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Include="NLog.config">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </None>
     <None Include="Package.StoreAssociation.xml" />
     <Content Include="Assets\QuickLook\quicklook_icon_black.png" />
     <Content Include="Assets\WSL\alpine.svg" />
@@ -466,6 +469,12 @@
     <PackageReference Include="Newtonsoft.Json">
       <Version>12.0.3</Version>
     </PackageReference>
+    <PackageReference Include="NLog">
+      <Version>4.6.8</Version>
+    </PackageReference>
+    <PackageReference Include="NLog.Schema">
+      <Version>4.6.8</Version>
+    </PackageReference>
   </ItemGroup>
   <ItemGroup>
     <SDKReference Include="WindowsDesktop, Version=10.0.18362.0">

Files/NLog.config

@@ -0,0 +1,14 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
+      xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+
+    <targets>
+        <target name="logfile" xsi:type="File" fileName="${var:LogPath}\debug.txt" concurrentWrites="true" />
+        <target name="logconsole" xsi:type="Console" />
+    </targets>
+
+    <rules>
+        <logger name="*" minlevel="Debug" writeTo="logconsole" />
+        <logger name="*" minlevel="Debug" writeTo="logfile" />
+    </rules>
+</nlog>