Can the content filter regex backtracks be optimized?
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 30/100
- Issue 类型
- 缺陷
- 描述清晰度
- 需要澄清
- 活跃度
- 停滞
- 技术栈
- php
- 领域
- performance
调研方向
没有指定文件或测试。从调用 preg_replace_callback() 的内容过滤路径开始，使用大约 518K 的 JSON 负载重现故障，并跟踪输出缓冲区的处理方式。完成的标准是，大型 post_content 值能够完成处理，且没有输出缺失或无法解释的错误。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- PHP
- 星标
- 26
- 派生
- 8
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
这个仓库没有索引到贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
littlebizzy/force-https 的其他 Issue
-
难度 3/5 1-2 天 新手友好度 38/100
littlebizzy/force-https#11 · 3 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 55/100
-
难度 1/5 1 小时以内 新手友好度 25/100
-
难度 2/5 1-3 小时 新手友好度 52/100
littlebizzy/force-https#4 · 2 条评论 ·
查看 littlebizzy/force-https 的全部 Issue
相似的 Issue
-
jira-created
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 1/5 1 小时以内 新手友好度 70/100
nunomaduro/phpinsights#745 ·
-
status/awaiting_triage
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 1/5 1 小时以内 新手友好度 90/100
WordPress/plugin-check#1486 ·
-
enhancement
难度 2/5 1-3 小时 新手友好度 88/100
az-digital/az_quickstart#6019 ·