Merge pull request #829 from sitkoru/modules/sentry/default-tags

feat(sentry): set default tags with app info
sitkoru · Apr 27, 2023 · 9ef504f · 9ef504f
2 parents e301ad2 + 9e50df6
commit 9ef504f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Sitko.Core.App/IApplicationContext.cs b/src/Sitko.Core.App/IApplicationContext.cs
@@ -74,7 +74,7 @@ private ApplicationOptions GetApplicationOptions()
 
         if (string.IsNullOrEmpty(applicationOptions.Environment))
         {
-            applicationOptions.Version = AspNetEnvironmentName;
+            applicationOptions.Environment = AspNetEnvironmentName;
         }
 
         ConfigureApplicationOptions(applicationOptions);

diff --git a/src/Sitko.Core.Sentry/SentryModule.cs b/src/Sitko.Core.Sentry/SentryModule.cs
@@ -21,6 +21,10 @@ public void PostConfigureHostBuilder(IApplicationContext context, IHostBuilder h
                     o.Dsn = startupOptions.Dsn;
                     o.Debug = startupOptions.EnableDebug;
                     o.TracesSampleRate = startupOptions.TracesSampleRate;
+                    o.DefaultTags.Add("ServiceId", context.Id.ToString());
+                    o.DefaultTags.Add("Service", context.Name);
+                    o.DefaultTags.Add("Environment", context.Environment);
+                    o.DefaultTags.Add("Version", context.Version);
                     startupOptions.ConfigureSentryOptions?.Invoke(context, o, startupOptions);
                 });
             });