".tgz" files somtimes decompressed in error.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- php
- Área
- networking
Línea de trabajo
Empieza en Requests.php, en la rama de descompresión protegida por la cabecera content-encoding de la respuesta. Reproduce el problema con el ejemplo .tgz vinculado de Bitbucket e inspecciona el tratamiento del valor "none". Se considera terminado cuando los archivos con content-encoding "none" no se descomprimen y el archivo descargado sigue siendo adecuado para verificar la suma de comprobación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- PHP
- Estrellas
- 3.6k
- Forks
- 503
- Merge medio
- 5 d 10 h
- PR fusionados (30 d)
- 3
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de WordPress/Requests
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
-
Release checklist for 2.0.22AbiertoType: meta
Dificultad 1/5 Menos de una hora Aptitud para principiantes 25/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Review which characters should be `trim`med for cookie and header dataPosiblemente ocupada @asllanmaciel la tomó hace 51 días. AbiertoStatus: help wanted Type: testing/chores/QA
Dificultad 4/5 3-5 días Aptitud para principiantes 62/100
-
testSNISupport no longer verifies SNIQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. AbiertoComponent: Transports Status: help wanted Type: testing/chores/QA
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Todos los issues de WordPress/Requests
Issues similares
-
Captured slideshow pass overwrites the route path, aborting compose with a rebind() TypeErrorAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Automattic/blocks-engine#5003 ·
Los mantenedores suelen responder en 1 día
-
from-cloud mythic tracked
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
MythicalLTD/FeatherPanel#459 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[Bug]: PostgreSQL backend: Queue.remove() / Job.remove() never emit the QueueEvents "removed" eventPosiblemente ocupada @1AbdullahButt la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
taskforcesh/bullmq#5085 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
shukiv/jabali-panel#2423 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100