fix(err): stackframes: skip empty list, get context per-event (#26530)

PostHog · Nov 29, 2024 · b1810ea · b1810ea
1 parent ce66669
commit b1810ea
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 7 deletions.
diff --git a/frontend/src/lib/components/Errors/ErrorDisplay.tsx b/frontend/src/lib/components/Errors/ErrorDisplay.tsx
@@ -28,13 +28,8 @@ function StackTrace({
     showAllFrames: boolean
 }): JSX.Element | null {
     const { stackFrameRecords } = useValues(stackFrameLogic)
-    const { loadFromRawIds } = useActions(stackFrameLogic)
     const displayFrames = showAllFrames ? frames : frames.filter((f) => f.in_app)
 
-    useEffect(() => {
-        loadFromRawIds(frames.map(({ raw_id }) => raw_id))
-    }, [frames, loadFromRawIds])
-
     const panels = displayFrames.map(
         ({ raw_id, source, line, column, resolved_name, lang, resolved, resolve_failure }, index) => {
             const record = stackFrameRecords[raw_id]
@@ -122,8 +117,19 @@ function FrameContextLine({
 }
 function ChainedStackTraces({ exceptionList }: { exceptionList: ErrorTrackingException[] }): JSX.Element {
     const hasAnyInApp = exceptionList.some(({ stacktrace }) => stacktrace?.frames?.some(({ in_app }) => in_app))
-
     const [showAllFrames, setShowAllFrames] = useState(!hasAnyInApp)
+    const { loadFromRawIds } = useActions(stackFrameLogic)
+
+    useEffect(() => {
+        const frames: ErrorTrackingStackFrame[] = exceptionList.flatMap((e) => {
+            const trace = e.stacktrace
+            if (trace?.type === 'resolved') {
+                return trace.frames
+            }
+            return []
+        })
+        loadFromRawIds(frames.map(({ raw_id }) => raw_id))
+    }, [exceptionList, loadFromRawIds])
 
     return (
         <>

diff --git a/frontend/src/lib/components/Errors/stackFrameLogic.ts b/frontend/src/lib/components/Errors/stackFrameLogic.ts
@@ -29,6 +29,9 @@ export const stackFrameLogic = kea<stackFrameLogicType>([
                 loadFromRawIds: async ({ rawIds }) => {
                     const loadedRawIds = Object.keys(values.stackFrameRecords)
                     rawIds = rawIds.filter((rawId) => !loadedRawIds.includes(rawId))
+                    if (rawIds.length === 0) {
+                        return values.stackFrameRecords
+                    }
                     const { results } = await api.errorTracking.stackFrames(rawIds)
                     return mapStackFrameRecords(results, values.stackFrameRecords)
                 },

diff --git a/rust/cymbal/src/bin/run.sh b/rust/cymbal/src/bin/run.sh
@@ -3,5 +3,6 @@ export KAFKA_CONSUMER_TOPIC="exception_symbolification_events"
 export OBJECT_STORAGE_BUCKET="posthog"
 export OBJECT_STORAGE_ACCESS_KEY_ID="object_storage_root_user"
 export OBJECT_STORAGE_SECRET_ACCESS_KEY="object_storage_root_password"
+export ALLOW_INTERNAL_IPS="true"
 
-cargo run --bin cymbal
+RUST_LOG=info cargo run --bin cymbal