Skip to content

Commit 616b518

Browse files
committed
fix: memory leak in OTEL traces
- apply fix after claude review
1 parent b6d3c31 commit 616b518

1 file changed

Lines changed: 12 additions & 2 deletions

File tree

langfuse/_task_manager/media_manager.py

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,12 @@ def _process_data_recursively(data: Any, level: int) -> Any:
202202
seen.add(id(data))
203203

204204
try:
205-
return _process_data_recursively(data.model_dump(), level + 1)
205+
try:
206+
dumped = data.model_dump()
207+
except Exception:
208+
return data
209+
210+
return _process_data_recursively(dumped, level + 1)
206211
finally:
207212
seen.discard(id(data))
208213

@@ -214,7 +219,12 @@ def _process_data_recursively(data: Any, level: int) -> Any:
214219
seen.add(id(data))
215220

216221
try:
217-
return _process_data_recursively(data.dict(), level + 1)
222+
try:
223+
dumped = data.dict()
224+
except Exception:
225+
return data
226+
227+
return _process_data_recursively(dumped, level + 1)
218228
finally:
219229
seen.discard(id(data))
220230

0 commit comments

Comments
 (0)