BugFix: Skip validating and parsing commit lines early (#1108)

Signed-off-by: Wissam Abu Ahmad <xy.wissam@gmail.com>

Author: wissamir

prometheus_client/parser.py

@@ -308,6 +308,8 @@ def build_metric(name: str, documentation: str, typ: str, samples: List[Sample])
                 continue
             candidate_name, quoted = '', False
             if len(parts) > 2:
+                if parts[1] != 'TYPE' and parts[1] != 'HELP':
+                    continue
                 candidate_name, quoted = _unquote_unescape(parts[2])
                 if not quoted and not _is_valid_legacy_metric_name(candidate_name):
                     raise ValueError

tests/test_parser.py

@@ -120,6 +120,17 @@ def test_blank_lines_and_comments(self):
 """)
         self.assertEqualMetrics([CounterMetricFamily("a", "help", value=1)], list(families))
 
+
+    def test_comments_parts_are_not_validated_against_legacy_metric_name(self):
+        # https://github.com/prometheus/client_python/issues/1108
+        families = text_string_to_metric_families("""
+# A simple. comment line where third token cannot be matched against METRIC_NAME_RE under validation.py
+# 3565 12345/4436467 another random comment line where third token cannot be matched against METRIC_NAME_RE under validation.py
+""")
+        self.assertEqualMetrics([], list(families))
+
+
+
     def test_tabs(self):
         families = text_string_to_metric_families("""#\tTYPE\ta\tcounter
 #\tHELP\ta\thelp