From 9c4eb5841c9954c6f6dae1054858f8682c330c26 Mon Sep 17 00:00:00 2001 From: Behzad Shabani Date: Mon, 31 Dec 2018 12:52:21 +0330 Subject: [PATCH 1/2] Add error related helper --- helpers.go | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 helpers.go diff --git a/helpers.go b/helpers.go new file mode 100644 index 0000000..bcca789 --- /dev/null +++ b/helpers.go @@ -0,0 +1,15 @@ +package shark + +// PanicIfError panic exception if the given error is not nil +func PanicIfError(err error) { + if err != nil { + panic(err.Error()) + } +} + +// PanicIfErrCustomMsg panic with custom message if given error is not nil +func PanicIfErrorWithMessage(err error, message string) { + if err != nil { + panic(message) + } +} From 55eb72f207622a88f3eaffd162c9b79ff2fe9b63 Mon Sep 17 00:00:00 2001 From: Behzad Shabani Date: Mon, 31 Dec 2018 12:53:02 +0330 Subject: [PATCH 2/2] move panic reporter and renderer to middleware package to fix import cycle --- renderer.go => middleware/renderer.go | 2 +- sentry.go => middleware/reporter.go | 11 +---------- 2 files changed, 2 insertions(+), 11 deletions(-) rename renderer.go => middleware/renderer.go (98%) rename sentry.go => middleware/reporter.go (88%) diff --git a/renderer.go b/middleware/renderer.go similarity index 98% rename from renderer.go rename to middleware/renderer.go index 0f3d176..4a017d3 100644 --- a/renderer.go +++ b/middleware/renderer.go @@ -1,4 +1,4 @@ -package shark +package middleware import ( "fmt" diff --git a/sentry.go b/middleware/reporter.go similarity index 88% rename from sentry.go rename to middleware/reporter.go index a578f19..25f9a4f 100644 --- a/sentry.go +++ b/middleware/reporter.go @@ -1,4 +1,4 @@ -package shark +package middleware import ( "errors" @@ -90,12 +90,3 @@ func getRequestLogs(context context.Context) string { // the date should be logged by iris' Logger, so we skip them return fmt.Sprintf("%v %s %s %s", status, path, method, ip) } - -func CaptureRoutineException(exceptions exceptions.AggregatedRoutineException) { - // packet := raven.NewPacket( - // errString, - // raven.NewException(errors.New(errString), raven.NewStacktrace(2, 3, nil)), - // ) - // - // raven.Capture(packet, getCaptureTags(exception)) -}