WordPressVIPMinimum.Security.PHPFilterFunctions.MissingThirdParameter shouldn't report for SCRIPT_FILENAME,...
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par la règle WordPressVIPMinimum.Security.PHPFilterFunctions.MissingThirdParameter et le point d’entrée filter_input(INPUT_SERVER, ...) présenté dans l’extrait. Examinez les variables fournies par SAPI listées dans php-src/sapi/cli/php_cli.c, lignes 330-347, et confirmez que la règle ne signale plus ces cas tout en conservant son signalement existant pour les autres entrées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Bug Description
Since filter_input always acts on the original data, which can also not be modified in userland, all values that are specified by the SAPI itself are safe and do not need sanitizing.
These are:
https://github.com/php/php-src/blob/master/sapi/cli/php_cli.c#L330-L347
PHP_SELF
SCRIPT_NAME
SCRIPT_FILENAME
PATH_TRANSLATED
DOCUMENT_ROOT
For fpm these are provided by apache/nginx and no further validation is required, since PHP would not even process the request if any of those were invalid
Minimal Code Snippet
echo filter_input( INPUT_SERVER, 'DOCUMENT_ROOT' );
Error Code
WordPressVIPMinimum.Security.PHPFilterFunctions.MissingThirdParameter
Environment
VIPCS version | 3.0.1
Tested Against main branch?
- I have verified the issue still exists in the
mainbranch of VIPCS. - I have verified the issue still exists in the
developbranch of VIPCS.
- Langage dominant
- PHP
- Étoiles
- 261
- Forks
- 44
- Merge moyen
- 3 j 9 h
- PR mergées (30 j)
- 10
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Automattic/VIP-Coding-Standards
-
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
Automattic/VIP-Coding-Standards#876 ·
Les mainteneurs répondent en général sous 1 jour
-
Breaking Change Type: Maintenance
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Automattic/VIP-Coding-Standards#849 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
Automattic/VIP-Coding-Standards#823 ·
Les mainteneurs répondent en général sous 1 jour
-
Status: Awaiting feedback
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#754 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Type: Enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Automattic/VIP-Coding-Standards#731 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Automattic/VIP-Coding-Standards
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
openemr/openemr#14857 · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
type: enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
-
bug complete-verify customer-reported
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
MagnaCapax/PMSS#1199 ·
Les mainteneurs répondent en général sous 1 jour
-
Update CleryEdge API EndpointOuvertesafety.uiowa.edu
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
Les mainteneurs répondent en général sous 1 jour