Handle Apollo multipart failed-request inspection safely

未关闭
#5,987 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
45/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
kotlin
领域
api, testing

调研方向

首先阅读 sentry-apollo-4 和 sentry-apollo-5 的集成代码，跟踪对 multipart 和增量响应的失败请求 body 检查。添加有关流式行为和 body 保留的测试，包括跳过检查或不支持检查的情况。完成的标准是，multipart 处理不会对 body 进行缓冲、阻塞或消耗，同时 spans 和 breadcrumbs 仍得到保留。

由索引模型根据 Issue 内容生成。

描述

Bug Errors Java Platform: Java

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.

主要语言
Kotlin
星标
1.4k
派生
478
平均合并
3 天 2 小时
30 天内合并 PR
70

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

getsentry/sentry-java 的其他 Issue

查看 getsentry/sentry-java 的全部 Issue

相似的 Issue

更多 Kotlin Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。