bjarneriis / nlog.targets.sentry Goto Github PK

Add a property to the SentryTarget.SendLogEventInfoPropertiesAsTags.
When true, send as tags. when false (default state) send as extra.

The target uses the synchronous method calls. This can cause delays if no connection to sentry is available. It would be better to use asynchronous calls.

In addition having a queue for unsent messages would be a great improvement.

You shloud fix mistyped attribute 'dns' as 'dsn'.

CaptureException with multiple parameters was made obsolete in the latest versions of Raven, instead a Capture(SentryEvent) method should be used. Furthermore, Raven changed the signature of Capture method and takes string[] as the fifth argument.

When doing the override of the simple write method:

protected override void Write(LogEventInfo logEvent)

Then there is no need to catch exceptions, actually it will prevent NLog from reacting to these exceptions. Forexample if using a RetryingWrapper or a FallbackWrapper.