如何从中间件设置日志记录上下文?

How can I set up the logging context from middleware?

我想通过请求中的项目填充日志上下文,例如:r.Header.Get("X-Request-Id")。我假设我可以从中间件覆盖处理程序中的日志类型。虽然它似乎不起作用而且我不确定为什么!

package main

import (
    "fmt"
    "net/http"
    "os"

    "github.com/apex/log"
    "github.com/gorilla/mux"
)

// Assumption: handler is the shared state between the functions
type handler struct{ Log *log.Entry }

// New creates a handler for this application to co-ordinate shared resources
func New() (h handler) { return handler{Log: log.WithFields(log.Fields{"test": "FAIL"})} }

func (h handler) index(w http.ResponseWriter, r *http.Request) {
    h.Log.Info("Hello from the logger")
    fmt.Fprint(w, "YO")
}

func main() {
    h := New()

    app := mux.NewRouter()
    app.HandleFunc("/", h.index)
    app.Use(h.loggingMiddleware)

    if err := http.ListenAndServe(":"+os.Getenv("PORT"), app); err != nil {
        log.WithError(err).Fatal("error listening")
    }

}

func (h handler) loggingMiddleware(next http.Handler) http.Handler {
    return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
        h.Log = log.WithFields(log.Fields{"test": "PASS"})
        next.ServeHTTP(w, r)
    })
}

你能看出为什么 h.Log = log.WithFields(log.Fields{"test": "PASS"}) 似乎对 h.Log.Info("Hello from the logger") 没有任何影响吗?

你需要你的记录器是请求范围的。每次有新连接进入时,您都在为整个处理程序全局设置它,这意味着您要求数据竞争和通常不受欢迎的行为。

对于请求范围的上下文,context.Context embedded in the request is perfect. You can access it through the Context() and WithContext 方法。

示例:

var loggerKey = "Some unique key"

func (h handler) loggingMiddleware(next http.Handler) http.Handler {
    return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
        ctx := r.Context()
        ctx = context.WithValue(ctx, loggerKey, log.WithFields(log.Fields{"test": "PASS"}))
        next.ServeHTTP(w, r.WithContext(ctx)
    })
}

然后访问您的记录器:

func doSomething(r *http.Request) error {
    log, ok := r.Context().Value(loggerKey).(*log.Logger) // Or whatever type is appropriate
    if !ok {
        return errors.New("logger not set on context!")
    }
    // Do stuff...
}