Can the content filter regex backtracks be optimized?
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 30/100
- issue の種類
- バグ
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- php
- 領域
- performance
調査の方向性
ファイル名もテスト名も指定されていない。preg_replace_callback()を呼び出すcontent-filterのパスから始め、約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 にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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
-
Disable nag notices オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 52/100
littlebizzy/force-https#4 · コメント 2 件 ·
littlebizzy/force-https の issue をすべて見る
似ている issue
-
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 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
grokability/snipe-it#19688 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100