write_prometheus plugin: Remove invalid characers from the unit.

diff --git a/src/write_prometheus.c b/src/write_prometheus.c
index f4f28eb02a16c098c06bf232d58fc521a8230db0..fd663c4d0e6cfbd8db5b7f579d01a890fbad8ea8 100644 (file)
--- a/src/write_prometheus.c
+++ b/src/write_prometheus.c
@@ -263,7 +263,8 @@ void format_metric_family_name(strbuf_t *buf, metric_family_t const *fam) {
   if (unit != NULL) {
     strbuf_printf(buf, "_%s", unit->prometheus);
   } else if (fam->unit != NULL && fam->unit[0] != '{') {
-    strbuf_printf(buf, "_%s", fam->unit);
+    strbuf_print(buf, "_");
+    strbuf_print_restricted(buf, fam->unit, VALID_NAME_CHARS, '_');
   }
 
   if (fam->type == METRIC_TYPE_COUNTER) {

diff --git a/src/write_prometheus_test.c b/src/write_prometheus_test.c
index 61d1de5cfccd9f76a099d5a262096d7fc1f1ec56..386d1eabef3848412766937c18bc9e30191fa3d5 100644 (file)
--- a/src/write_prometheus_test.c
+++ b/src/write_prometheus_test.c
@@ -108,14 +108,15 @@ DEF_TEST(format_metric_family_name) {
           .unit = "%",
           .want = "storage_filesystem_utilization_percent",
       },
-      /* Not yet supported:
       {
           .name = "astro.light.speed",
           .type = METRIC_TYPE_GAUGE,
           .unit = "m/s",
+          .want = "astro_light_speed_m_s",
+          /* Not yet supported. Should be:
           .want = "astro_light_speed_meters_per_second",
+          */
       },
-      */
   };
 
   for (size_t i = 0; i < STATIC_ARRAY_SIZE(cases); i++) {