Capture all repeated `Cookie` and `Set-Cookie` header values
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 52/100
Piste de recherche
Commencez par inspecter les chemins existants d’extraction des cookies et d’accès aux headers dans les intégrations OkHttp, Ktor, Apollo 3 et Apollo 4, puis comparez la façon dont les autres SDK gèrent les headers répétés. Suivez la politique de cookies partagée et le comportement en l’absence de Data-Collection avant d’exécuter les tests d’intégration. Le travail est terminé lorsque les valeurs répétées de Cookie et Set-Cookie sont couvertes sans gestion dangereuse des virgules, que le filtrage reste efficace et que les valeurs malformées échouent de manière sécurisée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
The OkHttp, Ktor, and Apollo integrations capture only one Cookie or Set-Cookie header value when creating request and response contexts. Their current header accessors select a single value, so additional cookie fields are omitted from Sentry telemetry.
This is valid HTTP behavior. In particular, HTTP/2 allows clients to split the Cookie field into multiple header fields for compression. Responses also commonly contain multiple Set-Cookie fields.
This is follow-up completeness work from #5811 and supports #5666.
Check what other SDKs are doing before implementing.
Proposed solution
Make cookie extraction list-aware across OkHttp, Ktor, Apollo 3, and Apollo 4:
- Combine repeated request
Cookiefields with;before applying the effective cookie policy. - Process each response
Set-Cookiefield independently because each field represents one cookie and its attributes. - Do not comma-join or comma-split
Set-Cookievalues. A validExpiresattribute contains a comma. - Add shared core helpers or integration-specific adapters that apply the effective cookie policy to every value.
- Preserve valid
Set-Cookieattributes and fail closed for malformed input.
Preserve the existing absent-Data-Collection compatibility behavior for each integration.
Acceptance criteria
- OkHttp, Ktor, Apollo 3, and Apollo 4 capture all repeated cookie header values supported by their header APIs.
- Multiple request
Cookiefields are combined with;and filtered as one cookie string. - Multiple response
Set-Cookiefields are filtered independently without comma splitting or joining. - Built-in sensitive-cookie filtering and configured allow-list, deny-list, and off behavior apply to every cookie value.
- Malformed cookie values never bypass filtering.
- Integration tests cover at least two request
Cookiefields and two responseSet-Cookiefields where the underlying header API supports repeated values.
- Langage dominant
- Kotlin
- Étoiles
- 1.4k
- Forks
- 479
- Merge moyen
- 2 j 18 h
- PR mergées (30 j)
- 66
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de getsentry/sentry-java
-
Platform: Java
Difficulté 2/5 Une demi-journée Accessibilité débutants 76/100
getsentry/sentry-java#6161 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Bug Java Platform: Android Platform: Java
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
getsentry/sentry-java#6138 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Feature Java Platform: Java Spans
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
getsentry/sentry-java#5984 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Android Task Traces
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
getsentry/sentry-java#5376 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Android Docs Errors
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
getsentry/sentry-java#5375 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de getsentry/sentry-java
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
micronaut-projects/micronaut-core#13717 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
Ikiru change domainOuverteBug Domain changed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 70/100
keiyoushi/extensions-source#19690 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Missing repro Platform: Android
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
software-mansion/react-native-reanimated#10816 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
KazumaProject/JapaneseKeyboard#1125 ·
Les mainteneurs répondent en général sous 1 jour