diff --git a/src/middleware/error.middleware.ts b/src/middleware/error.middleware.ts index 33eca2c4df782312914915a4df04837087a5e0ee..cada33fe4fb86bc49a5231f495851bc5b518ed62 100644 --- a/src/middleware/error.middleware.ts +++ b/src/middleware/error.middleware.ts @@ -1,6 +1,9 @@ import { Response, Request, NextFunction } from 'express'; export const errorMiddleware = (error: Error, req: Request, res: Response, next: NextFunction) => { + // eslint-disable-next-line no-console + console.log(error); + delete error.stack; next(error); }; diff --git a/src/modules/stack/stack.controller.ts b/src/modules/stack/stack.controller.ts index b2d423cb4b949af3f861fed0d7e3c06c36b7d630..abd24b373b57d0cda65580b050a7bf4ead04867d 100644 --- a/src/modules/stack/stack.controller.ts +++ b/src/modules/stack/stack.controller.ts @@ -4,7 +4,6 @@ import { ApiOperationGet, ApiOperationPost, ApiPath } from 'swagger-express-ts'; import { StackService } from './stack.service'; import { StackResponse } from './stack.dto'; import { StackValidationService } from './validation'; -import { LoggerService } from '../../shared/logger'; import { StatusCodes } from '../../common/constants'; @ApiPath({ @@ -16,7 +15,6 @@ export class StackController { constructor( private readonly stackService: StackService, private readonly validationService: StackValidationService, - private readonly logger: LoggerService, ) {} @ApiOperationPost({ @@ -50,12 +48,6 @@ export class StackController { return res.status(StatusCodes.CREATED).json(result); } catch (e) { - this.logger.error({ - placement: `[${this.constructor.name}].add`, - error: e, - arguments: Array.from([req.body]), - }); - next(e); } } @@ -78,11 +70,6 @@ export class StackController { return res.status(StatusCodes.OK_RESPONSE).json(result); } catch (e) { - this.logger.error({ - placement: `[${this.constructor.name}].get`, - error: e, - }); - next(e); } } diff --git a/src/modules/ttl/ttl.controller.ts b/src/modules/ttl/ttl.controller.ts index 61908aadfc99926e5660619143e4f72fe20ad90a..4318024aa144d403c7a1a886e55a5806453843a6 100644 --- a/src/modules/ttl/ttl.controller.ts +++ b/src/modules/ttl/ttl.controller.ts @@ -4,7 +4,6 @@ import { ApiPath, ApiOperationPost, ApiOperationGet, ApiOperationDelete } from ' import { TtlService } from './ttl.service'; import { TtlResponse } from './ttl.dto'; import { TtlValidationService } from './validation'; -import { LoggerService } from '../../shared/logger'; import { StatusCodes } from '../../common/constants'; @ApiPath({ @@ -16,7 +15,6 @@ export class TtlController { constructor( private readonly ttlService: TtlService, private readonly validationService: TtlValidationService, - private readonly logger: LoggerService, ) {} @ApiOperationPost({ @@ -52,12 +50,6 @@ export class TtlController { return res.status(StatusCodes.CREATED).json(result); } catch (e) { - this.logger.error({ - placement: `[${this.constructor.name}].add`, - error: e, - arguments: Array.from([req.body]), - }); - next(e); } } diff --git a/src/shared/logger/index.ts b/src/shared/logger/index.ts deleted file mode 100644 index 7e387e79695cd169dd32c5413187cfc6e788f8ae..0000000000000000000000000000000000000000 --- a/src/shared/logger/index.ts +++ /dev/null @@ -1,2 +0,0 @@ -export * from './logger.service'; -export * from './logger.interfaces'; diff --git a/src/shared/logger/logger.interfaces.ts b/src/shared/logger/logger.interfaces.ts deleted file mode 100644 index a850417b4bb6407b6e637f533af63387d12b9585..0000000000000000000000000000000000000000 --- a/src/shared/logger/logger.interfaces.ts +++ /dev/null @@ -1,11 +0,0 @@ -export type LoggerMessage = - | string - | { - placement: string; - arguments?: any; - error?: Error | string; - }; - -export interface ILoggerError { - error(message: LoggerMessage): void; -} diff --git a/src/shared/logger/logger.service.ts b/src/shared/logger/logger.service.ts deleted file mode 100644 index 5d2a4863cdec5725d3e7ce1b1661cd5d16126adb..0000000000000000000000000000000000000000 --- a/src/shared/logger/logger.service.ts +++ /dev/null @@ -1,10 +0,0 @@ -import { Service } from 'typedi'; -import { ILoggerError, LoggerMessage } from './logger.interfaces'; - -@Service() -export class LoggerService implements ILoggerError { - error(message: LoggerMessage) { - // eslint-disable-next-line no-console - console.log(message); - } -} diff --git a/src/modules/stack/stack.controller.spec.ts b/src/tests/stack/stack.controller.spec.ts similarity index 94% rename from src/modules/stack/stack.controller.spec.ts rename to src/tests/stack/stack.controller.spec.ts index 7a5d7b1562cd638a2479a5260e8355c1bd054b26..d9077657caf467fd4d205bf7c196d77880e9e33e 100644 --- a/src/modules/stack/stack.controller.spec.ts +++ b/src/tests/stack/stack.controller.spec.ts @@ -1,8 +1,8 @@ import 'reflect-metadata'; import { Container } from 'typedi'; -import { StackController } from './stack.controller'; -import { StackService } from './stack.service'; -import { StackValidationService } from './validation'; +import { StackController } from '../../modules/stack/stack.controller'; +import { StackService } from '../../modules/stack/stack.service'; +import { StackValidationService } from '../../modules/stack/validation'; import { StatusCodes } from '../../common/constants'; const mockResponse = { diff --git a/src/modules/stack/stack.repository.spec.ts b/src/tests/stack/stack.repository.spec.ts similarity index 93% rename from src/modules/stack/stack.repository.spec.ts rename to src/tests/stack/stack.repository.spec.ts index b54f897841ee5211cb61b9c597871edd78cae70c..f1955d9dbcb0e05dd3c0c5a57842ec467e36c02f 100644 --- a/src/modules/stack/stack.repository.spec.ts +++ b/src/tests/stack/stack.repository.spec.ts @@ -1,7 +1,7 @@ import 'reflect-metadata'; import { Container } from 'typedi'; -import { StackModel } from './stack.model'; -import { StackRepository } from './stack.repository'; +import { StackModel } from '../../modules/stack/stack.model'; +import { StackRepository } from '../../modules/stack/stack.repository'; describe('StackRepository', () => { let stackModel; diff --git a/src/modules/stack/validation/stack.validation-service.spec.ts b/src/tests/stack/stack.validation-service.spec.ts similarity index 95% rename from src/modules/stack/validation/stack.validation-service.spec.ts rename to src/tests/stack/stack.validation-service.spec.ts index 6f8ac79adda64b74dcb0f0be372256469cbe96de..674b6ef8f890f65b0d2bde619d92f8dffa4ba2b7 100644 --- a/src/modules/stack/validation/stack.validation-service.spec.ts +++ b/src/tests/stack/stack.validation-service.spec.ts @@ -1,6 +1,6 @@ import 'reflect-metadata'; import { Container } from 'typedi'; -import { StackValidationService } from './stack.validation-service'; +import { StackValidationService } from '../../modules/stack/validation'; describe('StackValidationService', () => { let validateService: StackValidationService; diff --git a/src/modules/ttl/ttl.controller.spec.ts b/src/tests/ttl/ttl.controller.spec.ts similarity index 96% rename from src/modules/ttl/ttl.controller.spec.ts rename to src/tests/ttl/ttl.controller.spec.ts index bf80b8488a395d472c933886625272cec37534f1..3d6c3b7bdeefe68d72467d8734e3c34d47a52f8f 100644 --- a/src/modules/ttl/ttl.controller.spec.ts +++ b/src/tests/ttl/ttl.controller.spec.ts @@ -1,8 +1,8 @@ import 'reflect-metadata'; import { Container } from 'typedi'; -import { TtlController } from './ttl.controller'; -import { TtlService } from './ttl.service'; -import { TtlValidationService } from './validation'; +import { TtlController } from '../../modules/ttl/ttl.controller'; +import { TtlService } from '../../modules/ttl/ttl.service'; +import { TtlValidationService } from '../../modules/ttl/validation'; import { StatusCodes } from '../../common/constants'; const mockResponse = { diff --git a/src/modules/ttl/validation/ttl.validation-service.spec.ts b/src/tests/ttl/ttl.validation-service.spec.ts similarity index 96% rename from src/modules/ttl/validation/ttl.validation-service.spec.ts rename to src/tests/ttl/ttl.validation-service.spec.ts index d1588bf257793ba88d23e3be61a4717508c93c65..2a8acf41a9027d35019828ca11efe09d6ad69b58 100644 --- a/src/modules/ttl/validation/ttl.validation-service.spec.ts +++ b/src/tests/ttl/ttl.validation-service.spec.ts @@ -1,6 +1,6 @@ import 'reflect-metadata'; import { Container } from 'typedi'; -import { TtlValidationService } from './ttl.validation-service'; +import { TtlValidationService } from '../../modules/ttl/validation'; describe('TtlValidationService', () => { let validateService: TtlValidationService;