Can the content filter regex backtracks be optimized?
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 30/100
- Tipo de issue
- Bug
- Clareza
- Precisa de esclarecimento
- Status de atividade
- Estagnada
- Stack de tecnologia
- php
- Domínio
- performance
Direção de pesquisa
Nenhum arquivo ou teste é especificado. Comece pelo caminho do filtro de conteúdo que chama preg_replace_callback(), reproduza a falha com um payload JSON de aproximadamente 518K e rastreie como o buffer de saída é processado. A tarefa estará concluída quando valores grandes de post_content forem processados sem saída ausente ou erros inexplicáveis.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- PHP
- Estrelas
- 26
- Forks
- 8
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Guia de contribuição
Nenhum guia de contribuição indexado para este repositório
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de littlebizzy/force-https
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 38/100
littlebizzy/force-https#11 · 3 comentários ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 55/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 25/100
-
Disable nag notices Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 52/100
littlebizzy/force-https#4 · 2 comentários ·
Todas as issues de littlebizzy/force-https
Issues semelhantes
-
jira-created
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 70/100
nunomaduro/phpinsights#745 ·
-
status/awaiting_triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
az-digital/az_quickstart#6019 ·