ami/tl-vikunja
1
0
Fork 0
Code

fix: do not push nil errors to sentry

Browse Source
This commit is contained in:
kolaente 2024-04-13 21:46:07 +02:00
parent e9de7d8a24
commit 1460d212ee
No known key found for this signature in database
GPG Key ID: F40E70337AB24C9B
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/routes/routes.go
View File

@ -156,10 +156,18 @@ func setupSentry(e *echo.Echo) {
if hub != nil { if hub != nil {
hub.WithScope(func(scope *sentry.Scope) { hub.WithScope(func(scope *sentry.Scope) {
scope.SetExtra("url", c.Request().URL) scope.SetExtra("url", c.Request().URL)
if herr.Internal == nil {
hub.CaptureException(err)
} else {
hub.CaptureException(herr.Internal) hub.CaptureException(herr.Internal)
}
}) })
} else {
if herr.Internal == nil {
sentry.CaptureException(err)
} else { } else {
sentry.CaptureException(herr.Internal) sentry.CaptureException(herr.Internal)
}
log.Debugf("Could not add context for sending error '%s' to sentry", err.Error()) log.Debugf("Could not add context for sending error '%s' to sentry", err.Error())
} }
log.Debugf("Error '%s' sent to sentry", err.Error()) log.Debugf("Error '%s' sent to sentry", err.Error())