fix(auto_source_config): Handle when the frame is None (#85235)

armenzg · web-flow · commit 6c5715af9b30 · 2025-02-14T13:29:46.000-08:00
Fixes [SENTRY-3NS9](https://sentry.sentry.io/issues/6299550160/) Fixes [SENTRY-3NSA](https://sentry.sentry.io/issues/6299583637/)
diff --git a/src/sentry/issues/auto_source_code_config/stacktraces.py b/src/sentry/issues/auto_source_code_config/stacktraces.py
@@ -20,6 +20,9 @@ def get_frames_to_process(
     for stacktrace in stacktraces:
         frames = stacktrace["frames"]
         for frame in frames:
+            if frame is None:
+                continue
+
             if platform in PROCESS_ALL_FRAMES:
                 frames_to_process.append(frame)
 
diff --git a/tests/sentry/issues/auto_source_code_config/test_stacktraces.py b/tests/sentry/issues/auto_source_code_config/test_stacktraces.py
@@ -77,6 +77,7 @@ def test_get_frames_to_process(
     "frames, expected",
     [
         ([], []),
+        ([None], []),
         ([{"in_app": True}], []),
     ],
 )

Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ def test_get_frames_to_process(`
`77`	`77`	`"frames, expected",`
`78`	`78`	`[`
`79`	`79`	`([], []),`
	`80`	`+ ([None], []),`
`80`	`81`	`([{"in_app": True}], []),`
`81`	`82`	`],`
`82`	`83`	`)`