p-sw/nestlogged
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(nestlogged): merging logger injection logic of exception with guard and interceptor

Browse Source
This commit is contained in:
Shinwoo PARK 2025-03-30 03:29:01 +09:00
parent 828c74958a
commit 3ba771f8d0
1 changed files with 2 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
packages/nestlogged/src/logged/override.ts
View File

@ -70,9 +70,9 @@ export function overrideBuild<F extends Array<any>, R>(
}
} else {
// special, can access to request object
if (type === 'guard' || type === 'interceptor') {
if (type === 'guard' || type === 'interceptor' || type === 'exception') {
// args[0] == ExecutionContext
const ctx = args[0] as ExecutionContext;
const ctx = args[type === 'exception' ? 1 : 0] as ExecutionContext;
if (ctx.getType() !== 'http') {
injectedLogger.error(
'Cannot inject logger: Request type is not http',
@ -98,16 +98,6 @@ export function overrideBuild<F extends Array<any>, R>(
[name, key],
req[REQUEST_LOG_ID],
);
} else if (type === 'exception') {
const req = (args[1] as ArgumentsHost).switchToHttp().getRequest();
if (req[REQUEST_LOG_ID] === undefined) {
req[REQUEST_LOG_ID] = ScopedLogger.createScopeId();
}
args[metadatas.scopedLoggerInjectableParam] = ScopedLogger.fromRoot(
baseLogger,
[name, key],
req[REQUEST_LOG_ID],
);
} else if (type === 'route') {
// args[metadatas.scopedLoggerInjectableParam] is now Request object, thanks to code in @LoggedRoute!!!!
let req = args[metadatas.scopedLoggerInjectableParam];