Skip to content

Commit 91e84f2

Browse files
authored
feat: add userId to self stats and redact sensitive query params for requestUrl (#90)
1 parent 12cac40 commit 91e84f2

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/router.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import {type AppContext, REQUEST_ID_PARAM_NAME} from '@gravity-ui/nodekit';
1+
import {type AppContext, REQUEST_ID_PARAM_NAME, USER_ID_PARAM_NAME} from '@gravity-ui/nodekit';
22
import {Express, Router} from 'express';
33

44
import {cspMiddleware, getAppPresets} from './csp/middleware';
@@ -123,11 +123,13 @@ export function setupRoutes(ctx: AppContext, expressApp: Express, routes: AppRou
123123
service: 'self',
124124
action: req.routeInfo.handlerName || UNNAMED_CONTROLLER,
125125
responseStatus: res.statusCode,
126+
// TODO(DakEnviy): Add responseSize
126127
requestId: req.originalContext.get(REQUEST_ID_PARAM_NAME) || '',
127-
requestTime: req.originalContext.getTime(), //We have to use req.originalContext here to get full time
128+
requestTime: req.originalContext.getTime(), // We have to use req.originalContext here to get full time
128129
requestMethod: req.method,
129-
requestUrl: req.originalUrl,
130+
requestUrl: ctx.utils.redactSensitiveQueryParams(req.originalUrl),
130131
traceId: req.originalContext.getTraceId() || '',
132+
userId: req.originalContext.get(USER_ID_PARAM_NAME) || '',
131133
});
132134
}
133135
}

0 commit comments

Comments
 (0)