vbatts/registry
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #738 from stevvooe/fix-log-output

Browse source 
Do not replace logger when adding hooks
This commit is contained in:
Richard Scothern 2015-07-24 09:38:30 -07:00
commit 925335c1e8
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
registry/handlers/app.go
View file

@ -298,7 +298,14 @@ func (app *App) configureRedis(configuration *configuration.Configuration) {
// configureLogHook prepares logging hook parameters. // configureLogHook prepares logging hook parameters.
func (app *App) configureLogHook(configuration *configuration.Configuration) { func (app *App) configureLogHook(configuration *configuration.Configuration) {
logger := ctxu.GetLogger(app).(*log.Entry).Logger entry, ok := ctxu.GetLogger(app).(*log.Entry)
if !ok {
// somehow, we are not using logrus
return
}
logger := entry.Logger
for _, configHook := range configuration.Log.Hooks { for _, configHook := range configuration.Log.Hooks {
if !configHook.Disabled { if !configHook.Disabled {
switch configHook.Type { switch configHook.Type {
@ -318,7 +325,6 @@ func (app *App) configureLogHook(configuration *configuration.Configuration) {
} }
} }
} }
app.Context = ctxu.WithLogger(app.Context, logger)
} }
func (app *App) ServeHTTP(w http.ResponseWriter, r *http.Request) { func (app *App) ServeHTTP(w http.ResponseWriter, r *http.Request) {