Bug fixed

PrestaShopCorp · Aug 17, 2021 · 7353795 · 7353795
1 parent 08dd115
commit 7353795
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 10 deletions.
diff --git a/example/logger/src/example.api.ts b/example/logger/src/example.api.ts
@@ -13,7 +13,7 @@ async function bootstrap() {
   const app = await NestFactory.create(ExampleModule, {
     logger,
   });
-  app.useLogger(logger);
+  app.useLogger(createStackdriverLoggerTool(app as any));
   const configService = app.get(ConfigService);
   if (configService.get<string>('FORCE_REQUEST_TRACE') === 'true') {
     app.use(

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "nestjs-pino-stackdriver",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "description": "NestJS Pino logger with Stackdriver support",
   "author": "Jacob Dharandas Méndez <[email protected]>",
   "license": "MIT",

diff --git a/src/tools/create-logger.tool.ts b/src/tools/create-logger.tool.ts
@@ -23,10 +23,10 @@ export const createLoggerTool: (
       config = configOrApp.get(LoggerConfig);
       context = configOrApp.get(Context);
     } else {
-      const config = new LoggerConfig(configOrApp);
+      config = new LoggerConfig(configOrApp);
       context = new Context(
         `System-Logger-${v4()}`,
-        config.context,
+        config?.context || {},
         {} as RequestType,
       );
     }

diff --git a/src/tools/create-stackdriver-logger.tool.ts b/src/tools/create-stackdriver-logger.tool.ts
@@ -22,11 +22,12 @@ export const createStackdriverLoggerTool = (
   configOrApp: INestApplication | LoggerConfigType = {} as LoggerConfigType,
   contextName?: string,
 ) => {
-  if (!isNestApplication(configOrApp)) {
-    return createLoggerTool(
-      createStackdriverLoggerConfig(configOrApp),
-      contextName,
-    );
+  if (isNestApplication(configOrApp)) {
+    return createLoggerTool(configOrApp, contextName);
   }
-  return createLoggerTool(configOrApp, contextName);
+
+  return createLoggerTool(
+    createStackdriverLoggerConfig(configOrApp),
+    contextName,
+  );
 };