Can the content filter regex backtracks be optimized?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- php
- Lĩnh vực
- performance
Hướng nghiên cứu
Không có tệp hoặc bài kiểm thử nào được nêu tên. Bắt đầu từ đường dẫn bộ lọc nội dung gọi preg_replace_callback(), tái hiện lỗi với một payload JSON khoảng 518K và theo dõi cách bộ đệm đầu ra được xử lý. Hoàn thành khi các giá trị post_content lớn được xử lý xong mà không bị thiếu đầu ra hoặc phát sinh lỗi không thể giải thích.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- PHP
- Star
- 26
- Fork
- 8
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của littlebizzy/force-https
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
littlebizzy/force-https#11 · 3 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 55/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 25/100
-
Disable nag notices Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 52/100
littlebizzy/force-https#4 · 2 bình luận ·
Tất cả issue của littlebizzy/force-https
Issue tương tự
-
status/awaiting_triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
WordPress/plugin-check#1486 ·
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
az-digital/az_quickstart#6019 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
grokability/snipe-it#19688 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100