Merge pull request #173 from scolestock/patch-1

bruno-garcia · web-flow · commit 6b3c204128af · 2019-03-18T09:37:13.000+01:00
fix: Diagnostic logging param order in BackgroundWorker
diff --git a/src/Sentry/Internal/BackgroundWorker.cs b/src/Sentry/Internal/BackgroundWorker.cs
@@ -136,7 +136,7 @@ private static async Task WorkerAsync(
                         }
                         catch (Exception exception)
                         {
-                            options.DiagnosticLogger?.LogError("Error while processing event: {0}. #{1} in queue.", exception, @event.EventId, queue.Count);
+                            options.DiagnosticLogger?.LogError("Error while processing event {1}: {0}. #{2} in queue.", exception, @event.EventId, queue.Count);
                         }
                     }
                     else

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ private static async Task WorkerAsync(`
`136`	`136`	`}`
`137`	`137`	`catch (Exception exception)`
`138`	`138`	`{`
`139`		`- options.DiagnosticLogger?.LogError("Error while processing event: {0}. #{1} in queue.", exception, @event.EventId, queue.Count);`
	`139`	`+ options.DiagnosticLogger?.LogError("Error while processing event {1}: {0}. #{2} in queue.", exception, @event.EventId, queue.Count);`
`140`	`140`	`}`
`141`	`141`	`}`
`142`	`142`	`else`