Add the changed values to the comment that notifies the maintainer about an improvement
Chưa có ai nhận issue này.
Đá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
- 58/100
Hướng nghiên cứu
Find the code that generates the bot comment for improvement suggestions, then trace how it reads the changed advisory JSON. Update the comment behavior so changed fields show their old and new values, and verify that range changes are described accurately without treating removed last-known version data as deletion.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
When someone suggests an improvement, the bot comments on the pull request and tags the maintainer. The comment does not say what changed, so the maintainer has to read the diff of the JSON file.
For a range change that diff is easy to misread. The events in ranges stay the same. The only line that changes is database_specific.last_known_affected_version_range, which the form deletes once the range and the fixed event agree. So the change looks like someone deleted version data. That is what happened in #9713 and #9714, where I widened the affected range of two cryptography advisories. Both the maintainer and the automated review read the diff as deleting version data.
It would help if the bot comment listed the fields that changed, with the old and the new value, for example Affected versions: >= 42.0.0, <= 48.0.0 -> >= 42.0.0, < 49.0.0.
- Ngôn ngữ chính
- Không có dữ liệu ngôn ngữ
- Star
- 2.5k
- Fork
- 772
- Merge trung bình
- 3 ngày 15 giờ
- Pull request đã merge (30 ngày)
- 46
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 github/advisory-database
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
github/advisory-database#9255 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
github/advisory-database#9164 · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
github/advisory-database#8994 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
github/advisory-database#8898 · 4 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
github/advisory-database#8841 ·
Tất cả issue của github/advisory-database
Issue tương tự
-
blocklist removal
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Azure/azure-functions-docker#1257 ·
-
area:proxy bug security severity:low track:open-source
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
bug server
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
bytedance/UI-TARS-desktop#2009 · 1 bình luận ·