python / python/python-docs-es

Avoiding `pull_request_target` usage in workflows

Đang mở
#3,864 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

bug github_actions
Ngôn ngữ chính
Python
Star
371
Fork
413
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

One of this repositories workflow's uses pull_request_target:

https://github.com/python/python-docs-es/blob/9b99734eefaa1b21480066abc9a336dad45b95cd/.github/workflows/pr-comment.yml#L4

Please see the GitHub documentation for a list of risks associated with the target. I see the workflow is currently quite careful, but IMO it is still better to split it into two, one for building and one for posting.

It is now disallowed by actions/checkout (see blog post). We are also considering disabling it across the organisation (new feature), but it would break this workflow.

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

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu với .github/workflows/pr-comment.yml tại dòng 4 và đọc hướng dẫn của GitHub về việc sử dụng pull_request_target một cách an toàn, cùng với thay đổi checkout được liên kết. Tách riêng trách nhiệm build và đăng như đề xuất, sau đó xác minh rằng không workflow nào sử dụng pull_request_target và cả hai action bắt buộc vẫn hoạt động an toàn.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
github-actions
Lĩnh vực
ci-cd, security
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
55/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.