Skip to content

Commit cc82dff

Browse files
committed
Merge pull request #2 from jmblab/feature/nlog-rollbarsharp-internal-logger
Add NLog.RollbarSharp exceptions logging to internal logger
2 parents 30be327 + d64ca30 commit cc82dff

File tree

1 file changed

+18
-10
lines changed

1 file changed

+18
-10
lines changed

src/RollbarTarget.cs

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
using NLog.Common;
1+
using System;
2+
using NLog.Common;
23
using NLog.Layouts;
34
using NLog.Targets;
45
using RollbarSharp;
@@ -29,18 +30,25 @@ public RollbarTarget()
2930

3031
protected override void Write(LogEventInfo logEvent)
3132
{
32-
var client = CreateClient(logEvent);
33-
var level = ConvertLogLevel(logEvent.Level);
34-
var title = Title.Render(logEvent);
33+
try
34+
{
35+
var client = CreateClient(logEvent);
36+
var level = ConvertLogLevel(logEvent.Level);
37+
var title = Title.Render(logEvent);
3538

36-
var notice = logEvent.Exception != null
37-
? client.NoticeBuilder.CreateExceptionNotice(logEvent.Exception)
38-
: client.NoticeBuilder.CreateMessageNotice(logEvent.FormattedMessage);
39+
var notice = logEvent.Exception != null
40+
? client.NoticeBuilder.CreateExceptionNotice(logEvent.Exception)
41+
: client.NoticeBuilder.CreateMessageNotice(logEvent.FormattedMessage);
3942

40-
notice.Level = level;
41-
notice.Title = title;
43+
notice.Level = level;
44+
notice.Title = title;
4245

43-
client.Send(notice);
46+
client.Send(notice);
47+
}
48+
catch (Exception exception)
49+
{
50+
InternalLogger.Error(exception.ToString());
51+
}
4452
}
4553

4654
/// <summary>

0 commit comments

Comments
 (0)