Can the content filter regex backtracks be optimized?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 30/100
- Tipo di issue
- Bug
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- php
- Ambito
- performance
Direzione di ricerca
Non sono indicati file né test. Inizia dal percorso del filtro dei contenuti che chiama preg_replace_callback(), riproduci il problema con un payload JSON di circa 518K e traccia il modo in cui viene elaborato il buffer di output. Il lavoro è completato quando i valori grandi di post_content terminano senza output mancante né errori inspiegabili.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- PHP
- Stelle
- 26
- Fork
- 8
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di littlebizzy/force-https
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
littlebizzy/force-https#11 · 3 commenti ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 55/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 25/100
-
Disable nag notices Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 52/100
littlebizzy/force-https#4 · 2 commenti ·
Tutte le issue di littlebizzy/force-https
Issue simili
-
status/awaiting_triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
az-digital/az_quickstart#6019 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
grokability/snipe-it#19688 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100