getsentry / getsentry/sentry-java

Handle Apollo multipart failed-request inspection safely

未關閉
#5,987 2 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
Bug Errors Java Platform: Java
主要語言
Kotlin
星號
1.4k
分支
478
平均合併
3 天 4 小時
30 天內合併 PR
72

描述

## Context

The Apollo 4 and Apollo 5 integrations inspect HTTP response bodies to detect GraphQL `errors`. Multipart and incremental responses need separate handling so inspection does not buffer, block, or consume streaming bodies.

## Scope

- Define safe failed-request inspection for multipart/incremental responses in `sentry-apollo-4` and `sentry-apollo-5`.
- Preserve spans and breadcrumbs when body inspection is skipped or unsupported.
- Decide between skipping failed-request body inspection for multipart responses and parsing multipart parts for GraphQL errors.
- Add tests covering streaming behavior and body preservation.

WebSocket subscription instrumentation remains out of scope.

貢獻指南

開啟貢獻指南

研究方向

先閱讀 sentry-apollo-4 和 sentry-apollo-5 的整合程式碼,追蹤對 multipart 和增量回應的失敗請求 body 檢查。新增串流行為和 body 保留的測試,包括略過檢查或不支援檢查的情況。完成的條件是,multipart 處理不會緩衝、阻塞或消耗 body,同時 spans 和 breadcrumbs 仍獲得保留。

由索引模型根據 Issue 內容生成。

評估

技術堆疊
kotlin
領域
api, testing
Issue 類型
缺陷
難度
4/5
預估耗時
3-5 天
活躍度
活躍
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。