From d5e720c404688a2df7eef8f3c5209df84b8a60ef Mon Sep 17 00:00:00 2001 From: Keivan Beigi Date: Fri, 20 Jan 2017 20:16:34 -0800 Subject: [PATCH] include os name, runtime name in version tag for sentry --- .../Instrumentation/Sentry/SentryTarget.cs | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/src/NzbDrone.Common/Instrumentation/Sentry/SentryTarget.cs b/src/NzbDrone.Common/Instrumentation/Sentry/SentryTarget.cs index 6ec6fd1e4..74cc391c3 100644 --- a/src/NzbDrone.Common/Instrumentation/Sentry/SentryTarget.cs +++ b/src/NzbDrone.Common/Instrumentation/Sentry/SentryTarget.cs @@ -37,10 +37,10 @@ namespace NzbDrone.Common.Instrumentation.Sentry { Compression = true, Environment = RuntimeInfo.IsProduction ? "production" : "development", - Release = BuildInfo.Release + Release = BuildInfo.Release, + ErrorOnCapture = OnError }; - _client.ErrorOnCapture = OnError; _client.Tags.Add("osfamily", OsInfo.Os.ToString()); _client.Tags.Add("runtime", PlatformInfo.Platform.ToString().ToLower()); @@ -135,10 +135,14 @@ namespace NzbDrone.Common.Instrumentation.Sentry sentryEvent.Fingerprint.Add(logEvent.Exception.GetType().FullName); } + var osName = Environment.GetEnvironmentVariable("OS_NAME"); + var osVersion = Environment.GetEnvironmentVariable("OS_VERSION"); + var runTimeVersion = Environment.GetEnvironmentVariable("RUNTIME_VERSION"); - sentryEvent.Tags.Add("os_name", Environment.GetEnvironmentVariable("OS_NAME")); - sentryEvent.Tags.Add("os_version", Environment.GetEnvironmentVariable("OS_VERSION")); - sentryEvent.Tags.Add("runtime_version", Environment.GetEnvironmentVariable("RUNTIME_VERSION")); + + sentryEvent.Tags.Add("os_name", osName); + sentryEvent.Tags.Add("os_version", $"{osName} {osVersion}"); + sentryEvent.Tags.Add("runtime_version", $"{PlatformInfo.Platform} {runTimeVersion}"); _client.Capture(sentryEvent); }