getsentry / getsentry/sentry-java

Handle Apollo multipart failed-request inspection safely

Aberta
#5,987 2 comentários 0 reações 0 responsáveis Ver no GitHub
Bug Errors Java Platform: Java
Linguagem predominante
Kotlin
Estrelas
1.4k
Forks
478
Merge médio
2d 22h
PRs com merge (30d)
69

Descrição

## 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.

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece lendo o código de integração de sentry-apollo-4 e sentry-apollo-5 para rastrear a inspeção do body de requests com falha para multipart e respostas incrementais. Adicione testes para o comportamento de streaming e a preservação do body, incluindo casos em que a inspeção é ignorada ou não tem suporte. Considera-se concluído quando o tratamento de multipart não armazena em buffer, bloqueia nem consome o body, enquanto spans e breadcrumbs permanecem preservados.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
kotlin
Domínio
api, testing
Tipo de issue
Bug
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Ativa
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.