".tgz" files somtimes decompressed in error.
Đá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
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- php
- Lĩnh vực
- networking
Hướng nghiên cứu
Bắt đầu trong Requests.php, tại nhánh giải nén được bảo vệ bởi header content-encoding của response. Tái hiện vấn đề với mẫu .tgz Bitbucket được liên kết và kiểm tra cách xử lý giá trị "none". Hoàn tất khi các file có content-encoding "none" không bị giải nén và archive đã tải xuống vẫn phù hợp để xác minh checksum.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Requests.php will mistakenly decompress a .tgz file (or other gzipped files) downloaded from sites that pass a "content-encoding: none" header.
In the case of .tgz files, you end up with a misnamed, uncompressed tar file. Often manageable, but not if there needs to be an md5/sha1/etc checksum confirmation.
The decompress code is wrapped by if (isset($return->headers['content-encoding'])), so the decompress code fires even though the encoding is "none". Should there be a check for a proper encoding type, or at minimum, check for "none" before decompressing?
Ran into this with bitbucket. A sample file to demonstrate the issue: https://bitbucket.org/jerrm-bb/testdownload/downloads/outcnam-test.tgz
- Ngôn ngữ chính
- PHP
- Star
- 3.6k
- Fork
- 503
- Merge trung bình
- 5 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 3
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc 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 WordPress/Requests
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 73/100
-
Release checklist for 2.0.22Đang mởType: meta
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 25/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
Review which characters should be `trim`med for cookie and header dataCó thể đã có người làm @asllanmaciel đã nhận 51 ngày trước. Đang mởStatus: help wanted Type: testing/chores/QA
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 62/100
-
testSNISupport no longer verifies SNICó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởComponent: Transports Status: help wanted Type: testing/chores/QA
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
Tất cả issue của WordPress/Requests
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Include flows spanning the full report time windowCó thể đã có người làm @somethingwithproof đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
Cacti/plugin_flowview#306 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
priority:low type:bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
crazy-goat/workerman-bundle#1189 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Captured slideshow pass overwrites the route path, aborting compose with a rebind() TypeErrorĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Automattic/blocks-engine#5003 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
from-cloud mythic tracked
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
MythicalLTD/FeatherPanel#459 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày