Merge pull request #2854 from manishtomar/log-authed-name

Log authorized username
This commit is contained in:
Ryan Abrams 2019-03-01 14:45:43 -08:00 committed by GitHub
commit d3ddc3572c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -862,7 +862,7 @@ func (app *App) authorized(w http.ResponseWriter, r *http.Request, context *Cont
return err return err
} }
dcontext.GetLogger(ctx).Info("authorized request") dcontext.GetLogger(ctx, auth.UserNameKey).Info("authorized request")
// TODO(stevvooe): This pattern needs to be cleaned up a bit. One context // TODO(stevvooe): This pattern needs to be cleaned up a bit. One context
// should be replaced by another, rather than replacing the context on a // should be replaced by another, rather than replacing the context on a
// mutable object. // mutable object.