luulam/AFFiNE
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

feat: log all exceptions In HTTP context (#4149)

Browse Source
This commit is contained in:
X1a0t
2023-09-04 15:42:55 +08:00
committed by GitHub
parent 80195ca2a0
commit ed76417ba3
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
apps/server/src/middleware/exception-logger.ts
View File

@@ -9,11 +9,12 @@ import { Request, Response } from 'express';
import { REQUEST_ID } from '../constants'; import { REQUEST_ID } from '../constants';
@Catch(HttpException) @Catch()
export class ExceptionLogger implements ExceptionFilter { export class ExceptionLogger implements ExceptionFilter {
private logger = new Logger('ExceptionLogger'); private logger = new Logger('ExceptionLogger');
catch(exception: Error, host: ArgumentsHost) { catch(exception: Error, host: ArgumentsHost) {
// with useGlobalFilters, the context is always HTTP
const ctx = host.switchToHttp(); const ctx = host.switchToHttp();
const request = ctx.getRequest<Request>(); const request = ctx.getRequest<Request>();
const requestId = request?.header(REQUEST_ID); const requestId = request?.header(REQUEST_ID);
@@ -27,11 +28,11 @@ export class ExceptionLogger implements ExceptionFilter {
const response = ctx.getResponse<Response>(); const response = ctx.getResponse<Response>();
if (exception instanceof HttpException) { if (exception instanceof HttpException) {
response.json(exception.getResponse()); response.status(exception.getStatus()).json(exception.getResponse());
} else { } else {
response.status(500).json({ response.status(500).json({
message: exception.message,
statusCode: 500, statusCode: 500,
error: exception.message,
}); });
} }
} }