Can the content filter regex backtracks be optimized?
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 30/100
- Issue-Typ
- Bug
- Klarheit
- Muss geklärt werden
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- php
- Bereich
- performance
Rechercherichtung
Es ist keine Datei und kein Test benannt. Beginne beim Content-Filter-Pfad, der preg_replace_callback() aufruft, reproduziere den Fehler mit einer JSON-Payload von ungefähr 518K und verfolge, wie der Ausgabepuffer verarbeitet wird. Als erledigt gilt die Aufgabe, wenn große post_content-Werte ohne fehlende Ausgabe oder unerklärliche Fehler vollständig verarbeitet werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
I have an edge-case scenario but wanted to suggest an improvement because it "could" affect users some day.
I am applying the content filter as an output buffer so that I can also convert non-https links outside of the post content. When applied to a 518K JSON payload, the preg_replace_callback() function fails. I could likely address this by modifying the pcre backtrack/recursion limits on my server.
Your users may trigger a similar edge case only if they have a large post_content field - unlikely, but remotely possible and a nasty gremlin to detect as there are no errors to explain the missing output.
I would submit a patch if/when I raise my regex game.
- Vorherrschende Sprache
- PHP
- Sterne
- 26
- Forks
- 8
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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 littlebizzy/force-https
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100
littlebizzy/force-https#11 · 3 Kommentare ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 55/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 25/100
-
Disable nag notices Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 52/100
littlebizzy/force-https#4 · 2 Kommentare ·
Alle Issues in littlebizzy/force-https
Ähnliche Issues
-
status/awaiting_triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
az-digital/az_quickstart#6019 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
grokability/snipe-it#19688 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100