diff --git a/CHANGELOG.md b/CHANGELOG.md index f72cae5..f83a027 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,13 @@ ## Unreleased +## 1.20.2 / 2024-08-23 + +* [BUGFIX] promhttp: Unset Content-Encoding header when data is uncompressed. #1596 + +## 1.20.1 / 2024-08-20 + +* [BUGFIX] process-collector: Fixed unregistered descriptor error when using process collector with `PedanticRegistry` on linux machines. #1587 + ## 1.20.0 / 2024-08-14 * [CHANGE] :warning: go-collector: Remove `go_memstat_lookups_total` metric which was always 0; Go runtime stopped sharing pointer lookup statistics. #1577 @@ -10,7 +18,7 @@ * [FEATURE] promhttp: Add experimental support for `zstd` on scrape, controlled by the request `Accept-Encoding` header. #1496 * [FEATURE] api/v1: Add `WithLimit` parameter to all API methods that supports it. #1544 * [FEATURE] prometheus: Add support for created timestamps in constant histograms and constant summaries. #1537 -* [FEATURE] process-collectors: Add network usage metrics: `process_network_receive_bytes_total` and `process_network_transmit_bytes_total`. #1555 +* [FEATURE] process-collector: Add network usage metrics: `process_network_receive_bytes_total` and `process_network_transmit_bytes_total`. #1555 * [FEATURE] promlint: Add duplicated metric lint rule. #1472 * [BUGFIX] promlint: Relax metric type in name linter rule. #1455 * [BUGFIX] promhttp: Make sure server instrumentation wrapping supports new and future extra responseWriter methods. #1480 diff --git a/VERSION b/VERSION index 3989355..769e37e 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -1.20.0 +1.20.2 diff --git a/prometheus/collectors/dbstats_collector_test.go b/prometheus/collectors/dbstats_collector_test.go index 0698bb2..057e243 100644 --- a/prometheus/collectors/dbstats_collector_test.go +++ b/prometheus/collectors/dbstats_collector_test.go @@ -21,7 +21,7 @@ import ( ) func TestDBStatsCollector(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() { db := new(sql.DB) if err := reg.Register(NewDBStatsCollector(db, "db_A")); err != nil { diff --git a/prometheus/collectors/go_collector_latest_test.go b/prometheus/collectors/go_collector_latest_test.go index 98259f8..2f5440a 100644 --- a/prometheus/collectors/go_collector_latest_test.go +++ b/prometheus/collectors/go_collector_latest_test.go @@ -63,7 +63,7 @@ var memstatMetrics = []string{ } func TestGoCollectorMarshalling(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() reg.MustRegister(NewGoCollector( WithGoCollectorRuntimeMetrics(GoRuntimeMetricsRule{ Matcher: regexp.MustCompile("/.*"), @@ -80,7 +80,7 @@ func TestGoCollectorMarshalling(t *testing.T) { } func TestWithGoCollectorDefault(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() reg.MustRegister(NewGoCollector()) result, err := reg.Gather() if err != nil { @@ -100,7 +100,7 @@ func TestWithGoCollectorDefault(t *testing.T) { } func TestWithGoCollectorMemStatsMetricsDisabled(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() reg.MustRegister(NewGoCollector( WithGoCollectorMemStatsMetricsDisabled(), )) @@ -157,7 +157,7 @@ func TestGoCollectorAllowList(t *testing.T) { }, } { t.Run(test.name, func(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() reg.MustRegister(NewGoCollector( WithGoCollectorMemStatsMetricsDisabled(), WithGoCollectorRuntimeMetrics(test.rules...), @@ -219,7 +219,7 @@ func TestGoCollectorDenyList(t *testing.T) { }, } { t.Run(test.name, func(t *testing.T) { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() reg.MustRegister(NewGoCollector( WithGoCollectorMemStatsMetricsDisabled(), WithoutGoCollectorRuntimeMetrics(test.matchers...), @@ -242,7 +242,7 @@ func TestGoCollectorDenyList(t *testing.T) { } func ExampleGoCollector() { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() // Register the GoCollector with the default options. Only the base metrics, default runtime metrics and memstats are enabled. reg.MustRegister(NewGoCollector()) @@ -252,7 +252,7 @@ func ExampleGoCollector() { } func ExampleGoCollector_WithAdvancedGoMetrics() { - reg := prometheus.NewRegistry() + reg := prometheus.NewPedanticRegistry() // Enable Go metrics with pre-defined rules. Or your custom rules. reg.MustRegister( diff --git a/prometheus/process_collector.go b/prometheus/process_collector.go index efbc3ea..62a4e7a 100644 --- a/prometheus/process_collector.go +++ b/prometheus/process_collector.go @@ -140,6 +140,8 @@ func (c *processCollector) Describe(ch chan<- *Desc) { ch <- c.maxVsize ch <- c.rss ch <- c.startTime + ch <- c.inBytes + ch <- c.outBytes } // Collect returns the current state of all metrics of the collector. diff --git a/prometheus/process_collector_test.go b/prometheus/process_collector_test.go index 11e0540..0d62d41 100644 --- a/prometheus/process_collector_test.go +++ b/prometheus/process_collector_test.go @@ -37,7 +37,7 @@ func TestProcessCollector(t *testing.T) { t.Skipf("skipping TestProcessCollector, procfs not available: %s", err) } - registry := NewRegistry() + registry := NewPedanticRegistry() if err := registry.Register(NewProcessCollector(ProcessCollectorOpts{})); err != nil { t.Fatal(err) } diff --git a/prometheus/promhttp/http.go b/prometheus/promhttp/http.go index 2e0b9a8..e598e66 100644 --- a/prometheus/promhttp/http.go +++ b/prometheus/promhttp/http.go @@ -203,8 +203,10 @@ func HandlerForTransactional(reg prometheus.TransactionalGatherer, opts HandlerO defer closeWriter() - rsp.Header().Set(contentEncodingHeader, encodingHeader) - + // Set Content-Encoding only when data is compressed + if encodingHeader != string(Identity) { + rsp.Header().Set(contentEncodingHeader, encodingHeader) + } enc := expfmt.NewEncoder(w, contentType) // handleError handles the error according to opts.ErrorHandling diff --git a/prometheus/promhttp/http_test.go b/prometheus/promhttp/http_test.go index 0ed8fe3..3ad2d1d 100644 --- a/prometheus/promhttp/http_test.go +++ b/prometheus/promhttp/http_test.go @@ -267,7 +267,7 @@ func TestInstrumentMetricHandler(t *testing.T) { t.Errorf("got HTTP status code %d, want %d", got, want) } - if got, want := writer.Header().Get(contentEncodingHeader), string(Identity); got != want { + if got, want := writer.Header().Get(contentEncodingHeader), ""; got != want { t.Errorf("got HTTP content encoding header %s, want %s", got, want) }