diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 18241fef6..bc3baf1fe 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -52,6 +52,7 @@ jobs: run: | set -x go install honnef.co/go/tools/cmd/staticcheck@latest + go install github.com/nishanths/exhaustive/cmd/exhaustive@latest make static - name: Check gosec run: | diff --git a/Makefile b/Makefile index 9f8a5441f..f505cd446 100644 --- a/Makefile +++ b/Makefile @@ -22,6 +22,7 @@ dev-deps: ## Install developer dependencies @go install github.com/securego/gosec/v2/cmd/gosec@latest @go install honnef.co/go/tools/cmd/staticcheck@latest @go install github.com/deepmap/oapi-codegen/cmd/oapi-codegen@latest + @go install github.com/nishanths/exhaustive/cmd/exhaustive@latest deps: ## Install dependencies. @go mod download @@ -54,6 +55,7 @@ unused: dev-deps # Look for unused code static: dev-deps staticcheck ./... + exhaustive ./... generate: dev-deps go generate ./... diff --git a/internal/api/logout.go b/internal/api/logout.go index 2d5a2ecbf..ea5b871f7 100644 --- a/internal/api/logout.go +++ b/internal/api/logout.go @@ -48,6 +48,7 @@ func (a *API) Logout(w http.ResponseWriter, r *http.Request) error { return terr } + //exhaustive:ignore Default case is handled below. switch scope { case LogoutLocal: return models.LogoutSession(tx, s.ID) diff --git a/internal/observability/logging.go b/internal/observability/logging.go index 74a50c436..2e2595679 100644 --- a/internal/observability/logging.go +++ b/internal/observability/logging.go @@ -91,7 +91,7 @@ func setPopLogger(sql string) { } switch lvl { - case logging.Debug: + case logging.SQL, logging.Debug: l.Debug(s) case logging.Info: l.Info(s)