Can the content filter regex backtracks be optimized?
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 30/100
- Type d'issue
- Bug
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- php
- Domaine
- performance
Piste de recherche
Aucun fichier ni test n’est nommé. Commencez par le chemin du filtre de contenu qui appelle preg_replace_callback(), reproduisez l’échec avec une charge utile JSON d’environ 518K et suivez le traitement du tampon de sortie. Le travail est terminé lorsque les valeurs volumineuses de post_content s’achèvent sans sortie manquante ni erreurs inexpliquées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- PHP
- Étoiles
- 26
- Forks
- 8
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de littlebizzy/force-https
-
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
littlebizzy/force-https#11 · 3 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 55/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 25/100
-
Disable nag notices Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 52/100
littlebizzy/force-https#4 · 2 commentaires ·
Toutes les issues de littlebizzy/force-https
Issues similaires
-
status/awaiting_triage
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
az-digital/az_quickstart#6019 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
grokability/snipe-it#19688 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100