(#471) Fix logging

src/Lanceur.Infra/Managers/ExecutionManager.cs

@@ -9,6 +9,7 @@
 using System.Diagnostics;
 using Lanceur.Core.BusinessLogic;
 using Lanceur.Infra.LuaScripting;
+using Lanceur.Infra.Utils;
 
 namespace Lanceur.Infra.Managers
 {
@@ -97,17 +98,18 @@ private IEnumerable<QueryResult> ExecuteProcess(AliasQueryResult query)
                 WorkingDirectory = query.WorkingDirectory,
                 WindowStyle = query.StartMode.AsWindowsStyle(),
             };
-
+            using var __ = _logger.ScopeProcessStartInfo(psi);
             if (query.IsElevated || query.RunAs == SharedKernel.Constants.RunAs.Admin)
             {
                 psi.Verb = "runas";
                 _logger.LogInformation("Run {FileName} as ADMIN", query.FileName);
             }
-
-            _logger.BeginSingleScope("ProcessStartInfo", psi);
-            _logger.LogDebug("Executing process for alias {AliasName}", query.Name);
-            using var __ = Process.Start(psi);
-            return QueryResult.NoResult;
+
+            using (Process.Start(psi))
+            {
+                _logger.LogDebug("Executing process for alias {AliasName}", query.Name);
+                return QueryResult.NoResult;
+            }
         }
 
         private IEnumerable<QueryResult> ExecuteUwp(AliasQueryResult query)

src/Lanceur.Infra/Utils/LoggerMixins.cs

@@ -0,0 +1,25 @@
+using Lanceur.Infra.Logging;
+using Microsoft.Extensions.Logging;
+using System.Diagnostics;
+
+namespace Lanceur.Infra.Utils;
+
+public static class LoggerMixins
+{
+    #region Methods
+
+    public static IDisposable ScopeProcessStartInfo(this ILogger logger, ProcessStartInfo psi)
+    {
+        return logger.BeginSingleScope("ProcessStartInfo", new
+        {
+            psi.FileName,
+            psi.Verb,
+            psi.Arguments,
+            psi.UseShellExecute,
+            psi.WorkingDirectory,
+            psi.WindowStyle
+        });
+    }
+
+    #endregion Methods
+}