From ef1bb1e991782167ef2dc2c24e7d556561fa37ef Mon Sep 17 00:00:00 2001 From: mikefonted Date: Mon, 6 Mar 2017 10:07:05 +0100 Subject: [PATCH] Update MetricsEndpointResponse encoding to UTF-8 (without BOM) Prometheus was not taking metrics from this endpoint because of the UTF-8-BOM encoding. The issue is known at Prometheus repositories as seen here: - https://github.com/prometheus/docs/issues/542 - https://github.com/prometheus/common/pull/42 --- Src/Metrics.NET.Prometheus/PrometheusReportExtensions.cs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Src/Metrics.NET.Prometheus/PrometheusReportExtensions.cs b/Src/Metrics.NET.Prometheus/PrometheusReportExtensions.cs index 0b419b7..9d5b4fd 100644 --- a/Src/Metrics.NET.Prometheus/PrometheusReportExtensions.cs +++ b/Src/Metrics.NET.Prometheus/PrometheusReportExtensions.cs @@ -17,7 +17,8 @@ public static class PrometheusReportExtensions /// public static MetricsEndpointReports WithPrometheusEndpointReport(this MetricsEndpointReports config, string path = "/prometheus") { - return WithPrometheusEndpointReport(config, path, Encoding.ASCII); + Encoding utf8WithoutBom = new UTF8Encoding(false); + return WithPrometheusEndpointReport(config, path, utf8WithoutBom); } ///