ProperEscapingFunctionSniff doesn't catch misuse of wp_json_encode
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 damit, WordPressVIPMinimum/Sniffs/Security/ProperEscapingFunctionSniff.php zu lesen, und reproduziere die minimalen onerror-Beispiele aus dem Issue. Vergleiche, wie wp_json_encode im Vergleich zu esc_html behandelt wird; abgeschlossen ist die Aufgabe, wenn die Fehlverwendung in diesem Kontext den WordPressVIPMinimum.Security.ProperEscapingFunction-Fehler erzeugt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Bug Description
ProperEscapingFunctionSniff is a great addition to WPCS. It isn't comprehensive, though.
One specific example that I ran into is that it won't catch improper use of wp_json_encode() like it does with other functions.
Minimal Code Snippet
<!-- correctly flags esc_html as the wrong escaping function in this context -->
<img src=a onerror="<?php echo esc_html( $foo ); ?>" />
<!-- fails to catch that this is the wrong escaping function -->
<img src=a onerror="<?php echo wp_json_encode( $foo ); ?>" />
Error Code
WordPressVIPMinimum.Security.ProperEscapingFunction
- 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
-
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
-
Type: Enhancement
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
Automattic/VIP-Coding-Standards#731 ·
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