Skip to content

Commit d732f50

Browse files
committed
fix: improved log filtering logic
1 parent ffcb63c commit d732f50

File tree

3 files changed

+20
-10
lines changed

3 files changed

+20
-10
lines changed

Diff for: package-lock.json

+7-7
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@
5555
"archiver": "^5.3.1",
5656
"aws-cdk": "^2.55.0",
5757
"aws-cdk-lib": "^2.55.0",
58-
"aws-sdk": "^2.1275.0",
58+
"aws-sdk": "^2.1276.0",
5959
"cdk-assets": "^2.55.0",
6060
"constructs": "^10.1.190",
6161
"esbuild": "^0.16.7",

Diff for: packages/commons/tests/utils/InvocationLogs.ts

+12-2
Original file line numberDiff line numberDiff line change
@@ -64,12 +64,22 @@ export class InvocationLogs {
6464
}
6565

6666
/**
67-
* Return only logs from function, exclude START, END, and REPORT generated by Lambda service
67+
* Return the index of the log that contains END RequestId
68+
* @param logs
69+
* @returns {number} index of the log that contains END RequestId
70+
*/
71+
public static getEndLogIndex(logs: string[]): number {
72+
return logs.findIndex((log) => log.includes('END RequestId'));
73+
}
74+
75+
/**
76+
* Return only logs from function, exclude START, END, REPORT, and XRay log generated by Lambda service
6877
* @param levelToFilter level to filter
6978
* @returns Array of function logs
7079
*/
7180
public getFunctionLogs(levelToFilter?: LEVEL): string[] {
72-
let filteredLogs = this.logs.slice(1, -2);
81+
const endLogIndex = InvocationLogs.getEndLogIndex(this.logs);
82+
let filteredLogs = this.logs.slice(1, endLogIndex);
7383

7484
if (levelToFilter) {
7585
filteredLogs = filteredLogs.filter((log) => {

0 commit comments

Comments
 (0)