Capture all repeated `Cookie` and `Set-Cookie` header values
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 52/100
Hướng nghiên cứu
Bắt đầu bằng cách kiểm tra các đường dẫn hiện có để trích xuất cookie và truy cập header trong các tích hợp OkHttp, Ktor, Apollo 3 và Apollo 4, sau đó so sánh cách các SDK khác xử lý các header lặp lại. Theo dõi chính sách cookie dùng chung và hành vi khi thiếu Data-Collection trước khi chạy các bài kiểm thử tích hợp. Hoàn thành có nghĩa là các giá trị Cookie và Set-Cookie lặp lại được bao phủ mà không xử lý dấu phẩy không an toàn, việc lọc vẫn hiệu quả và các giá trị không hợp lệ fail closed.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Kotlin
- Star
- 1.4k
- Fork
- 478
- Merge trung bình
- 3 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 70
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getsentry/sentry-java
-
Bug Java Platform: Java
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
getsentry/sentry-java#6138 · 1 bình luận ·
-
Feature Java Platform: Java Spans
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-java#5984 · 1 bình luận ·
-
Android Task Traces
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-java#5376 · 1 bình luận ·
-
Android Docs Errors
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 64/100
getsentry/sentry-java#5375 · 1 bình luận ·
-
Android good first issue Task User Feedbacks
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/sentry-java#5354 · 1 bình luận ·
Tất cả issue của getsentry/sentry-java
Issue tương tự
-
Icon Requests Đang mởsupported
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
LawnchairLauncher/lawnicons#4100 ·
-
cr:missing-repro cr:platform:android platform:android repro-provided
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
software-mansion/react-native-screens#4699 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Good First Issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
ankidroid/Anki-Android#21942 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
libre-tube/LibreTube#8781 · 1 bình luận ·