BC-8093 - Check logging implementation (Part 2) (#28)

.env.default

@@ -13,5 +13,5 @@ WS_PORT=3345
 TLDRAW__WEBSOCKET_URL=ws://localhost:3345
 FEATURE_TLDRAW_ENABLED=true
 
-NEST_LOG_LEVEL=DEBUG
+NEST_LOG_LEVEL=debug
 EXIT_ON_ERROR=true

.env.test

@@ -16,5 +16,5 @@ TLDRAW__ASSETS_MAX_SIZE_BYTES=10485760
 TLDRAW__ASSETS_ALLOWED_MIME_TYPES_LIST="image/png,image/jpeg,image/gif,image/svg+xml"
 FEATURE_TLDRAW_ENABLED=true
 
-NEST_LOG_LEVEL=DEBUG
+NEST_LOG_LEVEL=debug
 EXIT_ON_ERROR=true

src/infra/logger/interfaces/logger.interface.ts

@@ -1,12 +1,5 @@
-export type RequestLoggingBody = {
+export interface RequestLoggingBody {
 	userId?: string;
 	request: { url: string; method: string; params: unknown; query: unknown };
 	error: unknown | undefined;
-};
-
-export interface ILogger {
-	http(message: RequestLoggingBody, context?: string): void;
-	log(message: unknown, context?: string): void;
-	warn(message: unknown, context?: string): void;
-	debug(message: unknown, context?: string): void;
 }

src/infra/logger/logger.config.ts

@@ -1,9 +1,20 @@
 import { Transform } from 'class-transformer';
-import { IsBoolean, IsString } from 'class-validator';
+import { IsBoolean, IsEnum } from 'class-validator';
+
+export enum LoggerLogLevel {
+	emerg = 'emerg',
+	alert = 'alert',
+	crit = 'crit',
+	error = 'error',
+	warning = 'warning',
+	notice = 'notice',
+	info = 'info',
+	debug = 'debug',
+}
 
 export class LoggerConfig {
-	@IsString()
-	public NEST_LOG_LEVEL!: string;
+	@IsEnum(LoggerLogLevel)
+	public NEST_LOG_LEVEL!: LoggerLogLevel;
 
 	@IsBoolean()
 	@Transform(({ value }) => value === 'true')

src/infra/logger/logger.spec.ts

@@ -12,12 +12,12 @@ describe('Logger', () => {
 	let processStdoutWriteSpy: jest.SpyInstance<
 		boolean,
 		[str: string | Uint8Array, encoding?: BufferEncoding | undefined, cb?: ((err?: Error) => void) | undefined],
-		any
+		unknown
 	>;
 	let processStderrWriteSpy: jest.SpyInstance<
 		boolean,
 		[str: string | Uint8Array, encoding?: BufferEncoding | undefined, cb?: ((err?: Error) => void) | undefined],
-		any
+		unknown
 	>;
 	let winstonLogger: DeepMocked<WinstonLogger>;
 

src/infra/logger/logger.ts

@@ -3,10 +3,9 @@ import { WINSTON_MODULE_PROVIDER } from 'nest-winston';
 import * as util from 'util';
 import * as winston from 'winston';
 import { RequestLoggingBody } from './interfaces/index.js';
-import { ILogger } from './interfaces/logger.interface.js';
 
 @Injectable({ scope: Scope.TRANSIENT })
-export class Logger implements ILogger {
+export class Logger {
 	/**
 	 * This Logger Service can be injected into every Class,
 	 * use setContext() with CustomProviderClass.name that will be added to every log.