".tgz" files somtimes decompressed in error.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- php
- Ambito
- networking
Direzione di ricerca
Inizia in Requests.php nel ramo di decompressione protetto dall’header content-encoding della risposta. Riproduci il problema con l’esempio .tgz collegato di Bitbucket e analizza la gestione del valore "none". Il lavoro è completato quando i file con content-encoding "none" non vengono decompressi e l’archivio scaricato rimane adatto alla verifica del checksum.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- PHP
- Stelle
- 3.6k
- Fork
- 503
- Merge medio
- 5g 10h
- PR unite (30g)
- 3
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di WordPress/Requests
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
-
Release checklist for 2.0.22ApertaType: meta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 25/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Review which characters should be `trim`med for cookie and header dataForse già presa @asllanmaciel l’ha presa 51 giorni fa. ApertaStatus: help wanted Type: testing/chores/QA
Difficoltà 4/5 3-5 giorni Idoneità per principianti 62/100
-
testSNISupport no longer verifies SNIForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. ApertaComponent: Transports Status: help wanted Type: testing/chores/QA
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
Tutte le issue di WordPress/Requests
Issue simili
-
Captured slideshow pass overwrites the route path, aborting compose with a rebind() TypeErrorAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Automattic/blocks-engine#5003 ·
I maintainer di solito rispondono entro 1 giorno
-
from-cloud mythic tracked
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
MythicalLTD/FeatherPanel#459 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Bug]: PostgreSQL backend: Queue.remove() / Job.remove() never emit the QueueEvents "removed" eventForse già presa @1AbdullahButt l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
taskforcesh/bullmq#5085 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
shukiv/jabali-panel#2423 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100