Add packageid to sentry context when using update-all

stackriot-labs · May 17, 2019 · 3cf2d7d · 3cf2d7d
1 parent 229fcbc
commit 3cf2d7d
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/AppGet/Update/UpdateService.cs b/src/AppGet/Update/UpdateService.cs
@@ -3,6 +3,7 @@
 using System.Linq;
 using System.Threading.Tasks;
 using AppGet.HostSystem;
+using AppGet.Infrastructure.Logging;
 using AppGet.Installers;
 using AppGet.Manifest;
 using AppGet.PackageRepository;
@@ -70,6 +71,8 @@ public async Task UpdateAllPackages(InstallInteractivityLevel interactivityLevel
             {
                 var update = toInstall[index];
 
+                SentryTarget.AddTag("packageid", update.PackageId);
+
                 try
                 {
                     _logger.Info("Installing update {0} of {1}", index + 1, toInstall.Count);
@@ -86,6 +89,10 @@ public async Task UpdateAllPackages(InstallInteractivityLevel interactivityLevel
                     _logger.Fatal(e, "An error occurred while updating {0}", update.PackageId);
                     failed++;
                 }
+                finally
+                {
+                    SentryTarget.AddTag("packageid", null);
+                }
             }
 
             Console.WriteLine();