Can the content filter regex backtracks be optimized?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Error
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- php
- Área
- performance
Línea de trabajo
No se especifica ningún archivo ni prueba. Empieza por la ruta del filtro de contenido que llama a preg_replace_callback(), reproduce el fallo con un payload JSON de aproximadamente 518K y rastrea cómo se procesa el búfer de salida. Se considera terminado cuando los valores grandes de post_content se completan sin que falte salida ni se produzcan errores inexplicables.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- PHP
- Estrellas
- 26
- Forks
- 8
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de littlebizzy/force-https
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
littlebizzy/force-https#11 · 3 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 25/100
-
Disable nag notices Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
littlebizzy/force-https#4 · 2 comentarios ·
Todos los issues de littlebizzy/force-https
Issues similares
-
status/awaiting_triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
az-digital/az_quickstart#6019 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
grokability/snipe-it#19688 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100