umputun / remark42 / 28477819781

84%

master: 84%

Pull #2097

github

Move request middlewares to middleware.go and drop chi/middleware

Swap the last chi middleware (middleware.Timeout) for the local timeout
helper, which removes the final use of go-chi/chi/v5/middleware — the
dependency is gone (the chi router itself stays until the router migration).

Consolidate all request-scoped middlewares into a new middleware.go:
timeout (moved from ssl.go) plus rejectAnonUser, matchSiteID, cacheControl,
apiCSPMiddleware, securityHeadersMiddleware, subscribersOnly, validEmailAuth
and rateLimiter (moved from rest.go). No behaviour change; pure relocation
plus the Timeout swap. go test -race, vet, golangci-lint and govulncheck
clean.

142 of 149 new or added lines in 2 files covered. (95.3%)

6390 of 7576 relevant lines covered (84.35%)

36.35 hits per line