Avoid closure for onFinish handlers

Change-type: patch

Author: Page-

src/collectors/api/collect.ts

@@ -77,9 +77,9 @@ export const collectAPIMetrics = (
 				? 'aborted'
 				: 'completed';
 			req._metrics_gatherer.labels.statusCode = res.statusCode || '';
-			onFinishFuncs.forEach((f) => {
+			for (const f of onFinishFuncs) {
 				f();
-			});
+			}
 		});
 		next();
 	};

test/api-collector.spec.ts

@@ -45,8 +45,8 @@ describe('API metrics', () => {
 			/api_bytes_written_bucket{le="\+Inf",state="completed",statusCode="200"} 1/,
 			/api_latency_milliseconds_count{state="completed",statusCode="200"} 1/,
 		];
-		metricsRegexps.forEach((re) => {
+		for (const re of metricsRegexps) {
 			expect(re.test(output)).to.be.true;
-		});
+		}
 	});
 });