proposal: notify when the request.Context is not used

[komuw]

Go is frequently used for servers. And as such http.Handler is used often.

Given the following code;

package main

import (
	"context"
	"net/http"
)
func foo(ctx context.Context, a, b int) {}

func myHandler(w http.ResponseWriter, r *http.Request) {
	foo(context.Background(), 2, 33)
}

func main() {}

I think contextcheck should error with a message like;
Non-inherited new context, use r.Context() instead

And to resolve the error, you would have to do:

foo(r.Context(), 2, 33)

[kkHAIKE]

good idea 😊

[kkHAIKE]

Done in new version, golangci/golangci-lint#3102