fix: The condition ~= "/health" ensures that the subsequent code inside the if block will not be executed for requests to the "/health" endpoint, which is typically used for health checks.

srajasimman · srajasimman · commit cce1102e73b5 · 2024-07-01T23:14:31.000+05:30
diff --git a/prometheus.conf b/prometheus.conf
@@ -16,8 +16,10 @@ init_worker_by_lua_block {
     metric_connections = prometheus:gauge("nginx_http_connections","Number of HTTP connections", {"state"})
 }
 log_by_lua_block {
-    metric_requests:inc(1, {ngx.var.server_name, ngx.var.status, ngx.var.request_method, ngx.var.cache_status})
-    metric_latency:observe(tonumber(ngx.var.request_time),{ngx.var.server_name})
+    if ngx.var.request_uri ~= "/health" then
+        metric_requests:inc(1, {ngx.var.server_name, ngx.var.status, ngx.var.request_method, ngx.var.cache_status})
+        metric_latency:observe(tonumber(ngx.var.request_time),{ngx.var.server_name})
+    end
 }
 header_filter_by_lua_block {
     ngx.header["server"] = nil

Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,10 @@ init_worker_by_lua_block {`
`16`	`16`	`metric_connections = prometheus:gauge("nginx_http_connections","Number of HTTP connections", {"state"})`
`17`	`17`	`}`
`18`	`18`	`log_by_lua_block {`
`19`		`- metric_requests:inc(1, {ngx.var.server_name, ngx.var.status, ngx.var.request_method, ngx.var.cache_status})`
`20`		`- metric_latency:observe(tonumber(ngx.var.request_time),{ngx.var.server_name})`
	`19`	`+ if ngx.var.request_uri ~= "/health" then`
	`20`	`+ metric_requests:inc(1, {ngx.var.server_name, ngx.var.status, ngx.var.request_method, ngx.var.cache_status})`
	`21`	`+ metric_latency:observe(tonumber(ngx.var.request_time),{ngx.var.server_name})`
	`22`	`+ end`
`21`	`23`	`}`
`22`	`24`	`header_filter_by_lua_block {`
`23`	`25`	`ngx.header["server"] = nil`