WIP

Author: dreamorosi

Diff for: packages/logger/package.json

@@ -81,7 +81,7 @@
     "url": "https://github.com/awslabs/aws-lambda-powertools-typescript/issues"
   },
   "dependencies": {
-    "@aws-lambda-powertools/commons": "^0.2.0",
+    "@aws-lambda-powertools/commons": "^0.6.0",
     "@types/aws-lambda": "^8.10.72",
     "lodash.clonedeep": "^4.5.0",
     "lodash.merge": "^4.6.2",

Diff for: packages/logger/tests/e2e/basicFeatures.middy.test.ts

@@ -98,18 +98,16 @@ describe(`logger E2E tests basic functionalities (middy) for runtime: ${runtime}
       }
     }, TEST_CASE_TIMEOUT);
 
-    it('should include cold metrics only on the first invocation', async () => {
+    it('should include cold start equal to TRUE only on the first invocation', async () => {
       const coldStartLogMessages = invocationLogs[0].getFunctionLogs(LEVEL.INFO);
       for (const message of coldStartLogMessages) {
         expect(message).toContain(`"cold_start":true`);
       }
 
-      // TODO: There is a bug that the 'cold_start' field leaks to all invocation. (#518) 
-      // Link to tracked issue: https://github.com/awslabs/aws-lambda-powertools-typescript/issues/518
-      // const normalLogMessages = invocationLogs[1].getFunctionLogs(LEVEL.INFO);
-      // for(const message of normalLogMessages) {
-      //   expect(message).not.toContain(`"cold_start":true`);
-      // }
+      const normalLogMessages = invocationLogs[1].getFunctionLogs(LEVEL.INFO);
+      for (const message of normalLogMessages) {
+        expect(message).not.toContain(`"cold_start":true`);
+      }
     }, TEST_CASE_TIMEOUT);
   });