8000 Merge branch 'prometheus:master' into master · HelloBroBro/client_python@7b7bc3d · GitHub
[go: up one dir, main page]

Skip to content

Commit 7b7bc3d

Browse files
authored
Merge branch 'prometheus:master' into master
2 parents eb71bfc + 4535ce0 commit 7b7bc3d

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

prometheus_client/metrics.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -705,6 +705,8 @@ def info(self, val: Dict[str, str]) -> None:
705705
if self._labelname_set.intersection(val.keys()):
706706
raise ValueError('Overlapping labels for Info metric, metric: {} child: {}'.format(
707707
self._labelnames, val))
708+
if any(i is None for i in val.values()):
709+
raise ValueError('Label value cannot be None')
708710
with self._lock:
709711
self._value = dict(val)
710712

tests/test_core.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -534,6 +534,7 @@ def test_info(self):
534534

535535
def test_labels(self):
536536
self.assertRaises(ValueError, self.labels.labels('a').info, {'l': ''})
537+
self.assertRaises(ValueError, self.labels.labels('a').info, {'il': None})
537538

538539
self.labels.labels('a').info({'foo': 'bar'})
539540
self.assertEqual(1, self.registry.get_sample_value('il_info', {'l': 'a', 'foo': 'bar'}))

0 commit comments

Comments
 (0)
0