From d9fd202c97748b5215881a2b46a3d9efa5fcf4e2 Mon Sep 17 00:00:00 2001 From: Max Lancaster Date: Fri, 16 Nov 2018 15:11:14 -0500 Subject: [PATCH] Set configured sentry logger in all cases. This fixes a bug in my previous PR where the logger is not getting set in both branches of if/else block in the SentrySink class. --- .../Serilog.Sinks.Sentry.AspNetCore.csproj | 2 +- src/Serilog.Sinks.Sentry/SentrySink.cs | 2 +- src/Serilog.Sinks.Sentry/Serilog.Sinks.Sentry.csproj | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Serilog.Sinks.Sentry.AspNetCore/Serilog.Sinks.Sentry.AspNetCore.csproj b/src/Serilog.Sinks.Sentry.AspNetCore/Serilog.Sinks.Sentry.AspNetCore.csproj index 6b9e281..6aad6f6 100644 --- a/src/Serilog.Sinks.Sentry.AspNetCore/Serilog.Sinks.Sentry.AspNetCore.csproj +++ b/src/Serilog.Sinks.Sentry.AspNetCore/Serilog.Sinks.Sentry.AspNetCore.csproj @@ -2,7 +2,7 @@ A Sentry sink for Serilog - 2.4.0 + 2.5.0 Oleg Shevchenko net451;netstandard2.0 true diff --git a/src/Serilog.Sinks.Sentry/SentrySink.cs b/src/Serilog.Sinks.Sentry/SentrySink.cs index 07ab181..92cd7a8 100644 --- a/src/Serilog.Sinks.Sentry/SentrySink.cs +++ b/src/Serilog.Sinks.Sentry/SentrySink.cs @@ -101,7 +101,6 @@ public void Emit(LogEvent logEvent) _sentryRequestFactory ?? new SentryRequestFactory(httpContext), _sentryUserFactory ?? new SentryUserFactory(httpContext)) { - Logger = _logger, Release = _release, Environment = _environment }; @@ -115,6 +114,7 @@ public void Emit(LogEvent logEvent) }; } + ravenClient.Logger = _logger; ravenClient.LogScrubber = _dataScrubber; ravenClient.Capture(sentryEvent); } diff --git a/src/Serilog.Sinks.Sentry/Serilog.Sinks.Sentry.csproj b/src/Serilog.Sinks.Sentry/Serilog.Sinks.Sentry.csproj index 73ba05f..f0d7218 100644 --- a/src/Serilog.Sinks.Sentry/Serilog.Sinks.Sentry.csproj +++ b/src/Serilog.Sinks.Sentry/Serilog.Sinks.Sentry.csproj @@ -2,7 +2,7 @@ A Sentry sink for Serilog - 2.4.0 + 2.5.0 Oleg Shevchenko net45;netstandard2.0 true