feat(observability): utilise new logger in stats-web

[forbesus]

ref #436

[forbesus]

would you check this pr? @ygrishajev

[forbesus]

all things are completed @ygrishajev

[ygrishajev]

so how does it work with process.env and config within logger?

[forbesus]

how about now? @ygrishajev

[ygrishajev]

so I have LOG_LEVEL=debug but it's not picked up by config in the browser. I think config needs to be provided to the LoggerService constructor.

[forbesus]

so I have LOG_LEVEL=debug but it's not picked up by config in the browser. I think config needs to be provided to the LoggerService constructor.

i am not sure what this meaning exactly
could you tell me more detail?

[ygrishajev]

@forbesus LOG_LEVEL is an env var

LOG_LEVEL: z.enum(["fatal", "error", "warn", "info", "debug", "trace"]).optional().default("info"),

which controls which logs are done. It is a great way to configure logger on the backend, but it wouldn't work in most frontend implementations.

So LoggerService should probably implement both env vars and config object as param in constructor.

[forbesus]

how about now? @ygrishajev

[forbesus]

please check again @ygrishajev

[forbesus]

how about now? @ygrishajev

[forbesus]

hello @ygrishajev i just fixed this issue
would you check again?
thanks

[ygrishajev]

@forbesus I'm getting this error on the backend

stats-web-1  |  ⨯ Error: Cannot find module 'pino'
stats-web-1  | Require stack:
stats-web-1  | - /app/apps/stats-web/.next/server/app/_not-found/page.js
stats-web-1  |     at Object.pino (/app/apps/stats-web/.next/server/app/_not-found/page.js:176:18)
stats-web-1  |     at __webpack_require__ (/app/apps/stats-web/.next/server/webpack-runtime.js:33:43)
stats-web-1  |     at eval (../../packages/logging/src/servicies/logger/logger.service.ts:7:62)
stats-web-1  |     at (ssr)/../../packages/logging/src/servicies/logger/logger.service.ts (/app/apps/stats-web/.next/server/app/_not-found/page.js:775:1)
stats-web-1  |     at __webpack_require__ (/app/apps/stats-web/.next/server/webpack-runtime.js:33:43)
stats-web-1  |     at eval (../../packages/logging/src/index.ts:5:90)
stats-web-1  |     at (ssr)/../../packages/logging/src/index.ts (/app/apps/stats-web/.next/server/app/_not-found/page.js:764:1)
stats-web-1  |     at __webpack_require__ (/app/apps/stats-web/.next/server/webpack-runtime.js:33:43)
stats-web-1  |     at eval (./src/app/error.tsx:9:79)
stats-web-1  |     at (ssr)/./src/app/error.tsx (/app/apps/stats-web/.next/server/app/_not-found/page.js:324:1)
stats-web-1  |     at Object.__webpack_require__ [as require] (/app/apps/stats-web/.next/server/webpack-runtime.js:33:43)
stats-web-1  |     at JSON.parse (<anonymous>)
stats-web-1  |     at JSON.parse (<anonymous>)

[forbesus]

moved to #516