Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ci): verify triggering actor on pull request from fork #1894

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

soonum
Copy link
Contributor

@soonum soonum commented Dec 18, 2024

If a contributor that open a Pull Request from a fork is not part of the repository collaborators, then the workflow using check_triggering_actor subworkflowwill exit with a failure. It could be re-run later by a collaborator who has a write access.

This allows reviewers to read the code proposition before running the CI, ensuring no secrets are leaked outside the repository.

If a contributor that open a Pull Request from a fork is not part
of the repository collaborators, then the workflow using
check_triggering_actor subworkflowwill exit with a failure.
It could be re-run later by a collaborator who has a write access.

This allows reviewers to read the code proposition before running
the CI, ensuring no secrets are leaked outside the repository.
@soonum soonum added the ci label Dec 18, 2024
@soonum soonum requested a review from mayeul-zama December 18, 2024 17:02
@soonum soonum self-assigned this Dec 18, 2024
@cla-bot cla-bot bot added the cla-signed label Dec 18, 2024
@soonum soonum merged commit 9195753 into main Dec 18, 2024
211 checks passed
@soonum soonum deleted the dt/ci/check_triggering_actor branch December 18, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants