ref(parser): limit max stack lines we parse

Author: JonasBa

packages/utils/src/stacktrace.ts

@@ -1,6 +1,6 @@
 import type { StackFrame, StackLineParser, StackLineParserFn, StackParser } from '@sentry/types';
 
-const STACKTRACE_LIMIT = 50;
+const STACKTRACE_FRAME_LIMIT = 50;
 
 /**
  * Creates a stack parser with the supplied line parsers
@@ -14,7 +14,10 @@ export function createStackParser(...parsers: StackLineParser[]): StackParser {
 
   return (stack: string, skipFirst: number = 0): StackFrame[] => {
     const frames: StackFrame[] = [];
-    for (const line of stack.split('\n').slice(skipFirst)) {
+    const lines = stack.split('\n');
+
+    for (let i = skipFirst; i < lines.length; i++) {
+      const line = lines[i];
       // Ignore lines over 1kb as they are unlikely to be stack frames.
       // Many of the regular expressions use backtracking which results in run time that increases exponentially with
       // input size. Huge strings can result in hangs/Denial of Service:
@@ -35,6 +38,10 @@ export function createStackParser(...parsers: StackLineParser[]): StackParser {
           break;
         }
       }
+
+      if (frames.length >= STACKTRACE_FRAME_LIMIT) {
+        break;
+      }
     }
 
     return stripSentryFramesAndReverse(frames);
@@ -79,7 +86,7 @@ export function stripSentryFramesAndReverse(stack: StackFrame[]): StackFrame[] {
 
   // The frame where the crash happened, should be the last entry in the array
   return localStack
-    .slice(0, STACKTRACE_LIMIT)
+    .slice(0, STACKTRACE_FRAME_LIMIT)
     .map(frame => ({
       ...frame,
       filename: frame.filename || localStack[0].filename,