mirror of
https://github.com/Unleash/unleash.git
synced 2024-11-01 19:07:38 +01:00
5a3bb1ffc3
Lots of work here, mostly because I didn't want to turn off the `noImplicitAnyLet` lint. This PR tries its best to type all the untyped lets biome complained about (Don't ask me how many hours that took or how many lints that was >200...), which in the future will force test authors to actually type their global variables setup in `beforeAll`. --------- Co-authored-by: Gastón Fournier <gaston@getunleash.io>
40 lines
1.3 KiB
TypeScript
40 lines
1.3 KiB
TypeScript
import * as responseTime from 'response-time';
|
|
import EventEmitter from 'events';
|
|
import { REQUEST_TIME } from '../metric-events';
|
|
import { IFlagResolver } from '../types/experimental';
|
|
import { InstanceStatsService } from 'lib/services';
|
|
|
|
// eslint-disable-next-line @typescript-eslint/naming-convention
|
|
const _responseTime = responseTime.default;
|
|
|
|
const appNameReportingThreshold = 1000;
|
|
|
|
export function responseTimeMetrics(
|
|
eventBus: EventEmitter,
|
|
flagResolver: IFlagResolver,
|
|
instanceStatsService: Pick<InstanceStatsService, 'getAppCountSnapshot'>,
|
|
): any {
|
|
return _responseTime((req, res, time) => {
|
|
const { statusCode } = res;
|
|
const pathname = req.route ? req.baseUrl + req.route.path : '(hidden)';
|
|
|
|
let appName: string | undefined;
|
|
if (
|
|
!flagResolver.isEnabled('responseTimeWithAppNameKillSwitch') &&
|
|
(instanceStatsService.getAppCountSnapshot('7d') ??
|
|
appNameReportingThreshold) < appNameReportingThreshold
|
|
) {
|
|
appName = req.headers['unleash-appname'] ?? req.query.appName;
|
|
}
|
|
|
|
const timingInfo = {
|
|
path: pathname,
|
|
method: req.method,
|
|
statusCode,
|
|
time,
|
|
appName,
|
|
};
|
|
eventBus.emit(REQUEST_TIME, timingInfo);
|
|
});
|
|
}
|