Skip to content

Commit

Permalink
Revert "emergency situation: we're not logging to rollrus while we're…
Browse files Browse the repository at this point in the history
… past the request quota"

This reverts commit c107eed.
  • Loading branch information
jesseduffield committed Jun 29, 2019
1 parent c107eed commit 235a47b
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions pkg/app/app.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ import (
"github.com/sirupsen/logrus"
)

// we hit the limit: putting this here for now
const HaveNotHitRollbarLimit = false

// App struct
type App struct {
closers []io.Closer
Expand Down Expand Up @@ -83,7 +80,7 @@ func newLogger(config config.AppConfigurer) *logrus.Entry {
// https://github.com/aybabtme/humanlog
log.Formatter = &logrus.JSONFormatter{}

if HaveNotHitRollbarLimit && config.GetUserConfig().GetString("reporting") == "on" {
if config.GetUserConfig().GetString("reporting") == "on" {
// this isn't really a secret token: it only has permission to push new rollbar items
hook := rollrus.NewHook("23432119147a4367abf7c0de2aa99a2d", environment)
log.Hooks.Add(hook)
Expand Down

0 comments on commit 235a47b

Please sign in to comment.