Link destination parsing accepts control characters (but should not)
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- tooling
Hướng nghiên cứu
Bắt đầu với việc tái hiện JavaScript API trong issue và lần theo trình phân tích cú pháp đích liên kết được viết thủ công mà nó thực thi. Sử dụng quy tắc về ký tự điều khiển trong đặc tả CommonMark làm hành vi mong đợi; hoàn thành khi ký tự điều khiển không được chấp nhận làm đích và href được hiển thị không còn được tạo ra.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
From the spec:
A link destination consists of either
[...]
- a nonempty sequence of characters that does not include ASCII space or control characters [...]
In https://github.com/commonmark/commonmark.js/commit/531fd57c0777896471d1f9773d40c506b28e9d38, it looks like the handwritten parser doesn't stop on control characters. Saw this while looking at the changes for the new spec.
This can be verified using the JS API on dingus:
new commonmark.HtmlRenderer().render(new commonmark.Parser().parse("[a](\x01)"))
Results in:
"<p><a href=\"%01\">a</a></p>
"
- Ngôn ngữ chính
- JavaScript
- Star
- 1.6k
- Fork
- 231
- Merge trung bình
- 2 ngày 17 giờ
- Pull request đã merge (30 ngày)
- 2
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 2 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 38/100
commonmark/commonmark.js#298 ·
-
Độ 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 ·
Tất cả issue của commonmark/commonmark.js
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
Deepak3699/Ai_Mentor#244 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
ci-install-db-tools stall-case tests flake: stalled apt-get can be killed before it logs its callĐang mởeffort:low model:light plan planner:opus-5-5 tests
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Bug 🐞
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
mozilla-mobile/firefox-ios#35986 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Hidden loading spinner keeps animating after connect, causing high idle CPU usage in FirefoxĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
-
bug(sight): the dashboard's text truncations split surrogate pairs and show broken charactersĐang mởcomponent:sight
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
agentic-os-org/ANOLISA#6738 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày