feat(linters): add new linter canonicalheader

Canonicalheader checks http header via http.CanonicalHeaderKey().

Author: lasiar

.golangci.reference.yml

@@ -2514,6 +2514,7 @@ linters:
     - asciicheck
     - bidichk
     - bodyclose
+    - canonicalheader
     - containedctx
     - contextcheck
     - copyloopvar
@@ -2627,6 +2628,7 @@ linters:
     - asciicheck
     - bidichk
     - bodyclose
+    - canonicalheader
     - containedctx
     - contextcheck
     - copyloopvar

go.mod

@@ -65,6 +65,7 @@ require (
 	github.com/kulti/thelper v0.6.3
 	github.com/kunwardeep/paralleltest v1.0.10
 	github.com/kyoh86/exportloopref v0.1.11
+	github.com/lasiar/canonicalheader v1.0.5
 	github.com/ldez/gomoddirectives v0.2.4
 	github.com/ldez/tagliatelle v0.5.0
 	github.com/leonklingele/grouper v1.1.2

go.sum

@@ -0,0 +1,19 @@
+package canonicalheader
+
+import (
+	"github.com/lasiar/canonicalheader"
+	"golang.org/x/tools/go/analysis"
+
+	"github.com/golangci/golangci-lint/pkg/goanalysis"
+)
+
+func New() *goanalysis.Linter {
+	a := canonicalheader.Analyzer
+
+	return goanalysis.NewLinter(
+		a.Name,
+		a.Doc,
+		[]*analysis.Analyzer{canonicalheader.Analyzer},
+		nil,
+	).WithLoadMode(goanalysis.LoadModeTypesInfo)
+}

pkg/golinters/canonicalheader/canonicalheader.go

@@ -0,0 +1,11 @@
+package canonicalheader_test
+
+import (
+	"testing"
+
+	"github.com/golangci/golangci-lint/test/testshared/integration"
+)
+
+func TestFromTestdata(t *testing.T) {
+	integration.RunTestdata(t)
+}

pkg/golinters/canonicalheader/canonicalheader_test.go

@@ -0,0 +1,19 @@
+//golangcitest:args -Ecanonicalheader
+package testdata
+
+import "net/http"
+
+func canonicalheader() {
+	v := http.Header{}
+
+	v.Get("Test-HEader")          // want `non-canonical header "Test-HEader", instead use: "Test-Header"`
+	v.Set("Test-HEader", "value") // want `non-canonical header "Test-HEader", instead use: "Test-Header"`
+	v.Add("Test-HEader", "value") // want `non-canonical header "Test-HEader", instead use: "Test-Header"`
+	v.Del("Test-HEader")          // want `non-canonical header "Test-HEader", instead use: "Test-Header"`
+	v.Values("Test-HEader")       // want `non-canonical header "Test-HEader", instead use: "Test-Header"`
+
+	v.Set("Test-Header", "value")
+	v.Add("Test-Header", "value")
+	v.Del("Test-Header")
+	v.Values("Test-Header")
+}

pkg/golinters/canonicalheader/testdata/canonicalheader.go

@@ -7,6 +7,7 @@ import (
 	"github.com/golangci/golangci-lint/pkg/golinters/asciicheck"
 	"github.com/golangci/golangci-lint/pkg/golinters/bidichk"
 	"github.com/golangci/golangci-lint/pkg/golinters/bodyclose"
+	"github.com/golangci/golangci-lint/pkg/golinters/canonicalheader"
 	"github.com/golangci/golangci-lint/pkg/golinters/containedctx"
 	"github.com/golangci/golangci-lint/pkg/golinters/contextcheck"
 	"github.com/golangci/golangci-lint/pkg/golinters/copyloopvar"
@@ -823,6 +824,12 @@ func (LinterBuilder) Build(cfg *config.Config) ([]*linter.Config, error) {
 			WithLoadForGoAnalysis().
 			WithURL("https://github.com/ykadowak/zerologlint"),
 
+		linter.NewConfig(canonicalheader.New()).
+			WithSince("v1.58.0").
+			WithPresets(linter.PresetStyle).
+			WithLoadForGoAnalysis().
+			WithURL("https://github.com/lasiar/canonicalHeader"),
+
 		// nolintlint must be last because it looks at the results of all the previous linters for unused nolint directives
 		linter.NewConfig(nolintlint.New(&cfg.LintersSettings.NoLintLint)).
 			WithSince("v1.26.0").