Validator::isBoolean() accepts whitespace-only values
Nadie ha tomado este issue todavía.
- #617 de @raphyabak — cerrado sin fusionar
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
Línea de trabajo
Comienza en Validator::isBoolean() e inspecciona su comprobación existente de cadenas vacías y la ruta de validación que usa filter_var(). Añade cobertura para valores que solo contengan espacios en blanco, preservando las formas recortadas aceptadas, como ' on '. La tarea estará terminada cuando se rechacen los espacios y las tabulaciones, los valores vacíos sigan fallando y los valores booleanos válidos continúen pasando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Validator::isBoolean() accepts a whitespace-only value as valid, while an empty string is correctly rejected. filter_var() with FILTER_VALIDATE_BOOLEAN trims its input before matching, so a value that is only spaces or tabs carries no boolean but still passes.
// FLAG=" " in the environment
$dotenv->required('FLAG')->isBoolean(); // passes, but should not
// FLAG= (empty) correctly fails
Only whitespace-only values slip through; ' on ' still validates via the filter's own trimming, and a genuinely empty value already fails. The fix is to reject a value that is empty after trimming in isBoolean(), mirroring the existing empty-string guard.
This is a behaviour change, since whitespace-only values pass validation today and would start failing at boot. It belongs in the next major rather than a 5.x release.
- Lenguaje dominante
- PHP
- Estrellas
- 13.5k
- Forks
- 666
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 vlucas/phpdotenv
-
Windows checkout fails on tests/fixtures/env/nul.env in v5.7.0Posiblemente ocupada @littfed la tomó hace 18 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Unterminated double-quoted value at end of input is silently droppedPosiblemente ocupada @YoussefMansour9 la tomó hace 38 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Todos los issues de vlucas/phpdotenv
Issues similares
-
Awaiting Triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
tui: PHP code blocks without `<?php` are not syntax highlightedPosiblemente ocupada @SwastikTripathi la tomó hace 1 día. Abiertoreproduced
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
anomalyco/opencode#58664 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 87/100
WordPress/phpunit-test-runner#364 ·
Los mantenedores suelen responder en 4 días
-
domain/framework Platform(Default) priority/high
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día