fix(metrics): more duration histogram buckets (#142)

* fix(metrics): more duration histogram buckets

Closes #135

* fix: use the same buckets as boxo

Co-authored-by: Daniel Norman <1992255+2color@users.noreply.github.com>

---------

Co-authored-by: Daniel Norman <1992255+2color@users.noreply.github.com>

CHANGELOG.md

@@ -11,16 +11,12 @@ The following emojis are used to highlight certain changes:
 * 🛠 - BREAKING CHANGE.  Action is required if you use this functionality.
 * ✨ - Noteworthy change to be aware of.
 
-## [Unreleased]
-
-### Added
-
-### Changed
-
-### Removed
+## [v1.3.1]
 
 ### Fixed
 
+- Added more buckets to the duration histogram metric to allow for tracking operations that take longer than 1 minute.
+
 ### Security
 
 ## [v1.3.0]

metrics.go

@@ -19,14 +19,17 @@ func registerVersionMetric(version string) {
 	m.Set(1)
 }
 
+// Duration histograms  use fixed definition here, as we don't want to break existing buckets if we need to add more.
+var defaultDurationHistogramBuckets = []float64{0.05, 0.1, 0.25, 0.5, 1, 2, 5, 10, 30, 60, 120, 240, 480, 960, 1920}
+
 // withHTTPMetrics collects metrics around HTTP request/response count, duration, and size
 // per specific handler. Allows us to track them separately for /ipns and /ipfs.
 func withHTTPMetrics(handler http.Handler, handlerName string) http.Handler {
 
 	opts := prometheus.HistogramOpts{
 		Namespace:   "ipfs",
 		Subsystem:   "http",
-		Buckets:     []float64{0.05, 0.1, 0.2, 0.5, 1, 2, 5, 10, 30, 60},
+		Buckets:     defaultDurationHistogramBuckets,
 		ConstLabels: prometheus.Labels{"handler": handlerName},
 	}