Improve sanitization options: making it pluggable
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
- 35/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
Hướng nghiên cứu
Đọc lib/html.js, đặc biệt là potentiallyUnsafe, và theo dõi cách HtmlInline và HtmlBlock hiện đang xử lý HTML thô. Xác định sanitizer hook và hành vi thay thế được mô tả trong issue, sau đó xác minh rằng các tag an toàn như table, tr và /table có thể được giữ lại, trong khi nội dung script bị từ chối.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
I would like to have more sanitization options for raw html in markdown. For example: i would like to allow raw html bits like <table>, <tr> and </table> but disallow evil things like <script>takeOverTheWorld();</script>. Right now: we can either allow or disallow all raw html. If we make the sanitation pluggable, the user can choose which sanitation library he/she wants to plug in.
The most straightforward way to make sensitization pluggable would be to provide a function that does what 'potentiallyUnsafe' in lib/html.js does, plus a sanitizeRawHtml function that could be used for HtmlInline and HtmlBlock instead of just always omitting a comment '<!-- raw HTML omitted -->'.
This would be a minimum modification from what we have. I will create a pull request for this.
- Ngôn ngữ chính
- JavaScript
- Star
- 1.6k
- Fork
- 234
- Merge trung bình
- 1 ngày 27 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 commonmark/commonmark.js
-
White space handling looks inconsistent between HTML block element vs HTML inline elementCó thể đã có người làm @mamicicekel đã nhận 15 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#303 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
commonmark/commonmark.js#287 · 1 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 20/100
commonmark/commonmark.js#275 ·
-
Delegate node walking to othersĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
commonmark/commonmark.js#267 · 3 bình luận ·
-
Inconsistent escape of image altĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
commonmark/commonmark.js#264 · 1 reaction ·
Tất cả issue của commonmark/commonmark.js
Issue tương tự
-
Correction: IMOU Cruiser 4MPĐang mởcorrection via-web
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
ch-bas/cctv-camera-database#416 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug effort:medium model:light tests
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
piskyRpapalo/preceptoros-web#231 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
type/bug type/docs type/enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
Redo-San/RedoSan-Authenticity#574 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Place search spins forever when nominatim failsCó thể đã có người làm @jgt87 đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
anvaka/city-roads#213 ·