Custom IniSet Sniff to allow for certain safelisted options
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 48/100
Rechercherichtung
Beginne mit dem Vergleich der referenzierten Implementierung in WordPress/Sniffs/PHP/IniSetSniff.php, insbesondere der Zeilen 55-65, mit der Struktur des VIP-Coding-Standards-Sniffs. Überprüfe, wie die Whitelist konfiguriert ist, und identifiziere die relevanten Sniff-Tests. Als abgeschlossen gilt die Änderung, wenn die drei aufgeführten Session-Optionen akzeptiert werden, während andere nicht erlaubte ini_set-Aufrufe weiterhin Verstöße bleiben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- PHP
- Sterne
- 261
- Forks
- 44
- Ø Merge
- 3 T. 9 Std.
- Gemergte PRs (30 T.)
- 10
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Automattic/VIP-Coding-Standards
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
Automattic/VIP-Coding-Standards#876 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Breaking Change Type: Maintenance
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
Automattic/VIP-Coding-Standards#849 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
Automattic/VIP-Coding-Standards#832 ·
Maintainer antworten meist innerhalb von 1 Tag
-
WordPress.WP.AlternativeFunctions.file_system_operations error messages cannot be fixed anymoreOffen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
Automattic/VIP-Coding-Standards#823 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Status: Awaiting feedback
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
Automattic/VIP-Coding-Standards#754 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Automattic/VIP-Coding-Standards
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
tech-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
landsman/adminer-desktop#206 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Explosm: Cyanide & Happiness failed with: https://explosm.net/comics/latest#comic resulted in 307Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
RSS-Bridge/rss-bridge#5154 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[Sync EN] calendar: document the ValueErrors thrown for invalid calendar IDs and dates (#6005)Offensync-en
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
iftechfoundation/ifdb#1415 ·
Maintainer antworten meist innerhalb von 1 Tag