Automattic / Automattic/VIP-Coding-Standards
Custom IniSet Sniff to allow for certain safelisted options
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Merge medio
- 19 min
- PR fusionados (30 d)
- 1
Descripción
## Describe the solution you'd like
VIPCS could benefit from having a custom IniSet sniff...similar to https://github.com/WordPress/WordPress-Coding-Standards/blob/41f5a9c66ff814863bc479fb52fd6cd1abc87e28/WordPress/Sniffs/PHP/IniSetSniff.php#L55-L65, but we want to be able to customize the whitelist property for the below values:
- session.cookie_httponly
- session.cookie_secure
- session.use_only_cookies
WP already manages sessions but we have some clients that want to control PHP sessions via custom code.
## What code should *not* be reported as a violation?
```
ini_set('session.cookie_httponly', true);
ini_set('session.cookie_secure', true);
ini_set('session.use_only_cookies', true);
```
## Additional context
https://github.com/WordPress/WordPress-Coding-Standards/issues/1993
Guía de contribución
Línea de trabajo
Comienza comparando la implementación referenciada de WordPress/Sniffs/PHP/IniSetSniff.php, especialmente las líneas 55-65, con la estructura del sniff de VIP-Coding-Standards. Verifica cómo está configurada la whitelist e identifica las pruebas relevantes del sniff. Se considera terminado cuando las tres opciones de sesión indicadas se aceptan, mientras que otras llamadas a ini_set no permitidas siguen siendo infracciones.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100