Add Service Unavailable logging

dBildungsplattform · Jan 13, 2025 · 55511c9 · 55511c9
1 parent 6b8d496
commit 55511c9
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/src/shared/error/global-exception.filter.spec.ts b/src/shared/error/global-exception.filter.spec.ts
@@ -48,6 +48,23 @@ describe('GlobalExceptionFilter', () => {
                     httpException.getResponse(),
                     httpException.getStatus(),
                 );
+                expect(loggerMock.crit).toHaveBeenCalledTimes(0);
+            });
+        });
+
+        describe('when filter catches HttpException Service Unavailable', () => {
+            it('should log request url with crit and pass it on to the http adapter', () => {
+                const httpException: HttpException = new HttpException('exception', 503);
+
+                sut.catch(httpException, argumentsHost);
+
+                expect(adapterImplMock.reply).toHaveBeenCalledTimes(1);
+                expect(adapterImplMock.reply).toHaveBeenCalledWith(
+                    responseMock,
+                    httpException.getResponse(),
+                    httpException.getStatus(),
+                );
+                expect(loggerMock.crit).toHaveBeenCalledTimes(1);
             });
         });
 

diff --git a/src/shared/error/global-exception.filter.ts b/src/shared/error/global-exception.filter.ts
@@ -36,6 +36,10 @@ export class GlobalExceptionFilter implements ExceptionFilter {
 
         if (exception instanceof Error) {
             if (exception instanceof HttpException) {
+                if (exception.getStatus() === 503) {
+                    const url: string = (ctx.getRequest() satisfies Request)?.url;
+                    this.logger.crit(`503 Service Unavailable for URL: ${url}`);
+                }
                 httpAdapter.reply(ctx.getResponse(), exception.getResponse(), exception.getStatus());
             } else if (exception instanceof DriverException) {
                 this.logger.crit(exception.message, exception.stack);