Cannot add Content-Length header to a multipart part
Maintainer thường phản hồi trong vòng 1 ngày
@alexzargarian23 đang làm issue này rồi.
Từ ngày 24/10/2026.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 72/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- kotlin
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu với MultipartBody.Part.create và kiểm tra yêu cầu Content-Length được trích dẫn trong issue, sau đó xem các bài test multipart lân cận. Xác nhận cách xử lý các header của part được cung cấp thủ công đối với các kiểu multipart như multipart/mixed. Hoàn thành khi factory chấp nhận header ở những nơi được phép và các bài test bao quát hành vi này.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In #7319 the Content-Length header ceased being auto-included in every multipart part. That is fine, but manually adding that header to a part throws an exception:
MultipartBody.Part.create(
headers = Headers.Builder().add("Content-Length", TEST_DATA.size.toString()).build(),
body = TEST_DATA.toRequestBody(),
) // crash - "Unexpected header: Content-Length"
The crash is due to this line in the MultipartBody.Part.create method:
require(headers?.get("Content-Length") == null) { "Unexpected header: Content-Length" }
I think that require was added because that header was being auto-included at the time. Since that is no longer the case but the require still exists, it's impossible to add a Content-Length header using the factory method.
I believe including Content-Length on a multipart part is out-of-spec if the request is of type multipart/form-data, but is within RFC specs for other types like multipart/mixed.
Tests showing it's not possible to manually include Content-Length: https://gist.github.com/jasonrobinson-gpsw/830a56b81be94a704fad9409d8326865
- Ngôn ngữ chính
- Kotlin
- Star
- 47.1k
- Fork
- 9.3k
- Merge trung bình
- 1 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 31
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 lysine-dev/okhttp
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
lysine-dev/okhttp#9761 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
build-ci
lysine-dev/okhttp#9815 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Consider `ForwardingRequestBody`Có thể đã có người làm @Amir-Golmoradi đã nhận 3 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 57/100
lysine-dev/okhttp#9800 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
lysine-dev/okhttp#9799 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 67/100
lysine-dev/okhttp#9796 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của lysine-dev/okhttp
Issue tương tự
-
Messages: Subscribe to the new communication websocket topicsCó thể đã có người làm @krusche đã nhận 1 ngày trước. Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
ls1intum/artemis-android#801 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
enhancement good first issue
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 85/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
software-mansion/enriched-markdown#1084 ·
Maintainer thường phản hồi trong vòng 2 ngày