From aa74b67a334fca5b26775a40674effa5f7cbf42e Mon Sep 17 00:00:00 2001 From: Trong Huu Nguyen Date: Tue, 28 Jul 2026 11:21:53 +0200 Subject: [PATCH] refactor: remove unreachable functions Both were reported by deadcode: retry.WithBase was never used, and PrometheusMiddleware.Initialize is a leftover from the chi-prometheus code this middleware was adapted from. --- internal/retry/retry.go | 6 ------ pkg/middleware/prometheus.go | 8 -------- 2 files changed, 14 deletions(-) diff --git a/internal/retry/retry.go b/internal/retry/retry.go index 6f2e946..f3adf83 100644 --- a/internal/retry/retry.go +++ b/internal/retry/retry.go @@ -16,12 +16,6 @@ type fibonacci struct { type Option func(*fibonacci) -func WithBase(d time.Duration) Option { - return func(f *fibonacci) { - f.base = d - } -} - func WithMax(d time.Duration) Option { return func(f *fibonacci) { f.max = d diff --git a/pkg/middleware/prometheus.go b/pkg/middleware/prometheus.go index a812e14..d867ae0 100644 --- a/pkg/middleware/prometheus.go +++ b/pkg/middleware/prometheus.go @@ -62,14 +62,6 @@ func Prometheus(provider string, buckets ...float64) *PrometheusMiddleware { return &m } -func (m *PrometheusMiddleware) Initialize(path, method string, code int) { - m.reqs.WithLabelValues( - strconv.Itoa(code), - method, - path, - ) -} - func (m *PrometheusMiddleware) Handler(next http.Handler) http.Handler { relevantPaths := map[string]bool{ paths.OAuth2 + paths.Login: true,