Test CI is failing for external contributors
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
Hướng nghiên cứu
Start by reading the existing test CI workflow and the linked GitHub Security Lab article, then inspect how external pull requests handle secrets and the coverage.xml artifact. Confirm the two-workflow approach in a test external-contributor scenario. Done means test CI remains safe and coverage.xml is published successfully for external contributors.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Currently, test CI is using the pull_request trigger. However, with this trigger the secrets are not populated when the PR author is external to the repository.
This is done on purpose for security reasons: https://securitylab.github.com/resources/github-actions-preventing-pwn-requests/
Many persons on the internet suggest to use pull_request_target, but according to a rapid oversight of github securitylab article, this is only a workaround with does not help with security at all.
The proper solution seems to separate the workflow in two:
- existing workflow will only publish the coverage.xml as an artifact
- a new workflow will be responsible to publish this coverage.xml
Seems pretty simple to implement, but to be confirmed (I skimmed through github article way too fast) and tested of course.
- Ngôn ngữ chính
- Python
- Star
- 1
- Fork
- 2
- 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 openzim/_python-bootstrap
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
openzim/_python-bootstrap#57 ·
-
enhancement
openzim/_python-bootstrap#54 · 1 bình luận · 2 người được giao ·
-
Workflow convention Đang mởquestion
openzim/_python-bootstrap#53 · 1 người được giao ·
-
Move `.pre-commit.yaml` to hatch Đang mởenhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 25/100
openzim/_python-bootstrap#51 · 3 bình luận ·
-
What about nested logs Đang mởenhancement
openzim/_python-bootstrap#48 · 2 người được giao ·
Tất cả issue của openzim/_python-bootstrap
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
xinnan-tech/xiaozhi-fde-talk#263 ·
-
rules
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
huggingface/Repo2RLEnv#163 · 1 bình luận ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
huggingface/sentence-transformers#4074 ·
-
comp/dashboard invalid P3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
NousResearch/hermes-agent#121143 ·