Removed the ability for the config reader to read but not write the p…

…ort for prometheus (#697) * Removed the ability for the config reader to read but not write the port for prometheus * This test dont test this
yandex · Oct 2, 2024 · 2615441 · 2615441
1 parent f6f37b0
commit 2615441
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/docker/copy/config.conf b/docker/copy/config.conf
@@ -19,9 +19,6 @@ log_session yes
 log_query no
 log_stats yes
 stats_interval 60
-log_general_stats_prom yes
-log_route_stats_prom no
-promhttp_server_port 7777
 
 workers "auto"
 resolvers 1

diff --git a/sources/config_reader.c b/sources/config_reader.c
@@ -2643,6 +2643,12 @@ static int od_config_reader_parse(od_config_reader_t *reader,
 			continue;
 		}
 		case OD_LPROMHTTP_PORT: {
+#ifndef PROMHTTP_FOUND
+			od_config_reader_error(
+				reader, &token,
+				"promhttp_server_port read failed PROMHTTP_FOUND not set");
+			goto error;
+#endif
 			int port;
 			if (!od_config_reader_number(reader, &port))
 				goto error;