From f576cd46098fdece654dd98a4bd1faf2775ec19b Mon Sep 17 00:00:00 2001 From: KenTandrian Date: Thu, 12 Jun 2025 00:59:23 +0700 Subject: [PATCH] resolve comments --- apps/api/src/app/app.module.ts | 2 +- apps/api/src/middlewares/html-template.middleware.ts | 12 ++++++++---- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/apps/api/src/app/app.module.ts b/apps/api/src/app/app.module.ts index 8bc9c21fd..ae6d2f40a 100644 --- a/apps/api/src/app/app.module.ts +++ b/apps/api/src/app/app.module.ts @@ -136,7 +136,7 @@ import { UserModule } from './user/user.module'; providers: [I18nService] }) export class AppModule implements NestModule { - configure(consumer: MiddlewareConsumer) { + public configure(consumer: MiddlewareConsumer) { consumer.apply(HtmlTemplateMiddleware).forRoutes('*'); } } diff --git a/apps/api/src/middlewares/html-template.middleware.ts b/apps/api/src/middlewares/html-template.middleware.ts index 206e1ebb4..5cf353e9a 100644 --- a/apps/api/src/middlewares/html-template.middleware.ts +++ b/apps/api/src/middlewares/html-template.middleware.ts @@ -7,7 +7,7 @@ import { } from '@ghostfolio/common/config'; import { DATE_FORMAT, interpolate } from '@ghostfolio/common/helper'; -import { Injectable, NestMiddleware } from '@nestjs/common'; +import { Injectable, Logger, NestMiddleware } from '@nestjs/common'; import { format } from 'date-fns'; import { NextFunction, Request, Response } from 'express'; import * as fs from 'fs'; @@ -82,7 +82,7 @@ const locales = { export class HtmlTemplateMiddleware implements NestMiddleware { private indexHtmlMap: { [languageCode: string]: string } = {}; - constructor(private readonly i18nService: I18nService) { + public constructor(private readonly i18nService: I18nService) { try { this.indexHtmlMap = SUPPORTED_LANGUAGE_CODES.reduce( (map, languageCode) => ({ @@ -95,11 +95,15 @@ export class HtmlTemplateMiddleware implements NestMiddleware { {} ); } catch (error) { - console.error('Failed to load index.html files:', error); + Logger.error( + 'Failed to initialize index HTML map', + error, + 'HTMLTemplateMiddleware' + ); } } - use(request: Request, response: Response, next: NextFunction) { + public use(request: Request, response: Response, next: NextFunction) { const path = request.originalUrl.replace(/\/$/, ''); let languageCode = path.substr(1, 2);