Sentry should use CleanseLogMessage.

This commit is contained in:
Taloth Saldono 2017-04-07 20:42:39 +02:00
parent 94886e767b
commit c9d1807670
1 changed files with 10 additions and 3 deletions

View File

@ -111,11 +111,18 @@ namespace NzbDrone.Common.Instrumentation.Sentry
return;
}
var extras = logEvent.Properties.ToDictionary(x => x.Key.ToString(), x => x.Value.ToString());
var extras = logEvent.Properties.ToDictionary(x => x.Key.ToString(), x => CleanseLogMessage.Cleanse(x.Value.ToString()));
_client.Logger = logEvent.LoggerName;
string cleansedMessage = CleanseLogMessage.Cleanse(logEvent.Message);
string cleansedFormattedMessage = cleansedMessage;
var sentryMessage = new SentryMessage(logEvent.Message, logEvent.Parameters);
if (logEvent.Parameters != null)
{
cleansedFormattedMessage = CleanseLogMessage.Cleanse(string.Format(logEvent.Message, logEvent.Parameters));
}
var sentryMessage = new SentryMessage(cleansedFormattedMessage);
var sentryEvent = new SentryEvent(logEvent.Exception)
{
@ -126,7 +133,7 @@ namespace NzbDrone.Common.Instrumentation.Sentry
{
logEvent.Level.ToString(),
logEvent.LoggerName,
logEvent.Message
cleansedMessage
}
};