refactor: refactor logger (#107)

apps/api/src/app/inversify.config.ts

@@ -42,21 +42,13 @@ import {
   usdServiceSymbol,
 } from '@cowprotocol/services';
 import ms from 'ms';
-import pino from 'pino';
+import { Logger, logger } from '../logger';
 
 const DEFAULT_CACHE_VALUE_SECONDS = ms('2min') / 1000; // 2min cache time by default for values
 const DEFAULT_CACHE_NULL_SECONDS = ms('30min') / 1000; // 30min cache time by default for NULL values (when the repository isn't known)
 
 const CACHE_TOKEN_INFO_SECONDS = ms('24h') / 1000; // 24h
 
-// Configure the logger
-const logger = pino({
-  level: process.env.LOG_LEVEL || 'info',
-  formatters: {
-    level: (label) => ({ level: label }),
-  },
-});
-
 function getErc20Repository(cacheRepository: CacheRepository): Erc20Repository {
   return new Erc20RepositoryCache(
     new Erc20RepositoryViem(viemClients),
@@ -66,7 +58,7 @@ function getErc20Repository(cacheRepository: CacheRepository): Erc20Repository {
   );
 }
 
-function getCacheRepository(_apiContainer: Container): CacheRepository {
+function getCacheRepository(): CacheRepository {
   if (redisClient) {
     return new CacheRepositoryRedis(redisClient);
   }
@@ -150,10 +142,10 @@ function getApiContainer(): Container {
   const apiContainer = new Container();
 
   // Bind logger
-  apiContainer.bind<pino.Logger>('Logger').toConstantValue(logger);
+  apiContainer.bind<Logger>('Logger').toConstantValue(logger);
 
   // Repositories
-  const cacheRepository = getCacheRepository(apiContainer);
+  const cacheRepository = getCacheRepository();
   const erc20Repository = getErc20Repository(cacheRepository);
   const simulationRepository = getSimulationRepository();
   const tokenHolderRepository = getTokenHolderRepository(cacheRepository);

apps/api/src/logger.ts

@@ -0,0 +1,17 @@
+import pino from 'pino';
+
+export type Logger = pino.Logger;
+
+const loggerConfigEnv =
+  process.env.NODE_ENV === 'production'
+    ? {}
+    : {
+        transport: {
+          target: 'pino-pretty',
+        },
+      };
+
+export const logger = pino({
+  ...loggerConfigEnv,
+  level: process.env.LOG_LEVEL ?? 'info',
+});

apps/api/src/main.ts

@@ -1,20 +1,6 @@
 import Fastify from 'fastify';
 import { app } from './app/app';
-import pino from 'pino';
-
-const loggerConfigEnv =
-  process.env.NODE_ENV === 'production'
-    ? {}
-    : {
-        transport: {
-          target: 'pino-pretty',
-        },
-      };
-
-const logger = pino({
-  ...loggerConfigEnv,
-  level: process.env.LOG_LEVEL ?? 'info',
-});
+import { logger } from './logger';
 
 const host = process.env.HOST ?? 'localhost';
 const port = process.env.PORT ? Number(process.env.PORT) : 3001;