diff --git a/Gopkg.lock b/Gopkg.lock index 860587e7..a3ace871 100644 --- a/Gopkg.lock +++ b/Gopkg.lock @@ -105,14 +105,6 @@ revision = "a6af135bd4e28680facf08a3d206b454abc877a4" version = "v1.0.1" -[[projects]] - branch = "master" - digest = "1:d457d39e88f678ed14ac29517c3d74927a48dbc6a9f073fa241cf364a68cbe5c" - name = "github.com/heroku/rollrus" - packages = ["."] - pruneopts = "NUT" - revision = "fc0cef2ff331aebb24cd4e9ded7e20650f3d7006" - [[projects]] digest = "1:aaa38889f11896ee3644d77e17dc7764cc47f5f3d3b488268df2af2b52541c5f" name = "github.com/imdario/mergo" @@ -145,6 +137,22 @@ pruneopts = "NUT" revision = "02db52c7e406c7abec44c717a173c7715e4c1b62" +[[projects]] + branch = "master" + digest = "1:7429d1589cc1454e9e97ba48cb7c5025a085b32eae4d3163c517e3e5d6c7a7a0" + name = "github.com/jesseduffield/roll" + packages = ["."] + pruneopts = "NUT" + revision = "695be2e62b007ecef6644383f0ee9504c1f5cbb9" + +[[projects]] + branch = "master" + digest = "1:ec8982ff49a9ebdd7d7dfe320258ad19a07baafb5eeba7f41b31ee2072d1bba9" + name = "github.com/jesseduffield/rollrus" + packages = ["."] + pruneopts = "NUT" + revision = "7086c0b1b1bd21ef1f502e914ff2505a68c7d638" + [[projects]] branch = "master" digest = "1:e0306e92cef41d02af05a9f88395ec49c94f0255d54ca88ee3c02ae8f5806ee3" @@ -256,14 +264,6 @@ revision = "f35b8ab0b5a2cef36673838d662e249dd9c94686" version = "v1.2.2" -[[projects]] - branch = "master" - digest = "1:e42372d3f4921ec35df07f9b23239631e9d28580f7c1edcca212bc6daddc68fe" - name = "github.com/stvp/roll" - packages = ["."] - pruneopts = "NUT" - revision = "3627a5cbeaeaa68023abd02bb8687925265f2f63" - [[projects]] digest = "1:cd5ffc5bda4e0296ab3e4de90dbb415259c78e45e7fab13694b14cde8ab74541" name = "github.com/tcnksm/go-gitconfig" @@ -326,11 +326,11 @@ "github.com/docker/docker/client", "github.com/fatih/color", "github.com/go-errors/errors", - "github.com/heroku/rollrus", "github.com/imdario/mergo", "github.com/jesseduffield/asciigraph", "github.com/jesseduffield/gocui", "github.com/jesseduffield/pty", + "github.com/jesseduffield/rollrus", "github.com/jesseduffield/yaml", "github.com/mcuadros/go-lookup", "github.com/mgutz/str", diff --git a/Gopkg.toml b/Gopkg.toml index 683fce3d..8cad8078 100644 --- a/Gopkg.toml +++ b/Gopkg.toml @@ -48,3 +48,7 @@ [[constraint]] branch = "v2" name = "github.com/jesseduffield/yaml" + +[[constraint]] + branch = "master" + name = "github.com/jesseduffield/roll" diff --git a/pkg/log/log.go b/pkg/log/log.go index e33f93ed..d3daf328 100644 --- a/pkg/log/log.go +++ b/pkg/log/log.go @@ -6,8 +6,8 @@ import ( "os" "path/filepath" - "github.com/heroku/rollrus" "github.com/jesseduffield/lazydocker/pkg/config" + "github.com/jesseduffield/rollrus" "github.com/sirupsen/logrus" ) diff --git a/vendor/github.com/stvp/roll/LICENSE b/vendor/github.com/jesseduffield/roll/LICENSE similarity index 100% rename from vendor/github.com/stvp/roll/LICENSE rename to vendor/github.com/jesseduffield/roll/LICENSE diff --git a/vendor/github.com/stvp/roll/client.go b/vendor/github.com/jesseduffield/roll/client.go similarity index 98% rename from vendor/github.com/stvp/roll/client.go rename to vendor/github.com/jesseduffield/roll/client.go index 687343f2..284a3a88 100644 --- a/vendor/github.com/stvp/roll/client.go +++ b/vendor/github.com/jesseduffield/roll/client.go @@ -203,7 +203,8 @@ func (c *rollbarClient) send(item map[string]interface{}) (uuid string, err erro resp, err := http.Post(Endpoint, "application/json", bytes.NewReader(jsonBody)) if err != nil { - return "", err + // If something goes wrong it really does not matter + return "", nil } defer func() { io.Copy(ioutil.Discard, resp.Body) @@ -211,7 +212,8 @@ func (c *rollbarClient) send(item map[string]interface{}) (uuid string, err erro }() if resp.StatusCode != http.StatusOK { - return "", fmt.Errorf("Rollbar returned %s", resp.Status) + // If something goes wrong it really does not matter + return "", nil } // Extract UUID from JSON response diff --git a/vendor/github.com/stvp/roll/stack.go b/vendor/github.com/jesseduffield/roll/stack.go similarity index 100% rename from vendor/github.com/stvp/roll/stack.go rename to vendor/github.com/jesseduffield/roll/stack.go diff --git a/vendor/github.com/heroku/rollrus/LICENSE b/vendor/github.com/jesseduffield/rollrus/LICENSE similarity index 100% rename from vendor/github.com/heroku/rollrus/LICENSE rename to vendor/github.com/jesseduffield/rollrus/LICENSE diff --git a/vendor/github.com/heroku/rollrus/rollrus.go b/vendor/github.com/jesseduffield/rollrus/rollrus.go similarity index 95% rename from vendor/github.com/heroku/rollrus/rollrus.go rename to vendor/github.com/jesseduffield/rollrus/rollrus.go index 50dd225d..2d32b4e5 100644 --- a/vendor/github.com/heroku/rollrus/rollrus.go +++ b/vendor/github.com/jesseduffield/rollrus/rollrus.go @@ -1,4 +1,4 @@ -// Package rollrus combines github.com/stvp/roll with github.com/sirupsen/logrus +// Package rollrus combines github.com/jesseduffield/roll with github.com/sirupsen/logrus // via logrus.Hook mechanism, so that whenever logrus' logger.Error/f(), // logger.Fatal/f() or logger.Panic/f() are used the messages are // intercepted and sent to rollbar. @@ -22,9 +22,9 @@ import ( "os" "time" + "github.com/jesseduffield/roll" "github.com/pkg/errors" "github.com/sirupsen/logrus" - "github.com/stvp/roll" ) var defaultTriggerLevels = []logrus.Level{ @@ -37,7 +37,7 @@ var defaultTriggerLevels = []logrus.Level{ type Hook struct { roll.Client triggers []logrus.Level - ignoredErrors map[error]struct{} + ignoredErrors []error ignoreErrorFunc func(error) bool ignoreFunc func(error, map[string]string) bool @@ -78,12 +78,10 @@ func WithMinLevel(level logrus.Level) OptionFunc { } // WithIgnoredErrors is an OptionFunc that whitelists certain errors to prevent -// them from firing. +// them from firing. See https://golang.org/ref/spec#Comparison_operators func WithIgnoredErrors(errors ...error) OptionFunc { return func(h *Hook) { - for _, e := range errors { - h.ignoredErrors[e] = struct{}{} - } + h.ignoredErrors = append(h.ignoredErrors, errors...) } } @@ -120,7 +118,7 @@ func NewHookForLevels(token string, env string, levels []logrus.Level) *Hook { return &Hook{ Client: roll.New(token, env), triggers: levels, - ignoredErrors: make(map[error]struct{}), + ignoredErrors: make([]error, 0), ignoreErrorFunc: func(error) bool { return false }, ignoreFunc: func(error, map[string]string) bool { return false }, } @@ -179,8 +177,10 @@ func (r *Hook) Levels() []logrus.Level { // returned by Levels(). func (r *Hook) Fire(entry *logrus.Entry) error { trace, cause := extractError(entry) - if _, ok := r.ignoredErrors[cause]; ok { - return nil + for _, ie := range r.ignoredErrors { + if ie == cause { + return nil + } } if r.ignoreErrorFunc(cause) {