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

Continuously validate ghasum checksums at the job level #36

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

ericcornelissen
Copy link
Collaborator

Followup to #17
Relates to #2, #4

Summary

Remove the GitHub Actions workflow that continuously validates the ghasum checksums for all Actions in this project in favor of a per-job step that validates the ghasum checksums for that job only. This reduces makes jobs self-validating, which prevents potentially compromised jobs from executing while allowing other jobs to keep running.

The implementation leverages GitHub Actions context values to determine what job to validate, resulting in an identical step in all jobs. This makes it a good proof of concept for a ghasum action that can be used in any job.

Because of how this is implemented it is still NOT QUITE preventing running potentially compromised actions. In particular, because it requires this project's source code and correct Go version the actions/checkout and actions/setup-go (respectively) must be run first.

Remove the GitHub Actions workflow that continuously validates the
ghasum checksums for all Actions in this project in favor of a per-job
step that validates the ghasum checksums for that job only. This reduces
makes jobs self-validating, which prevents potentially compromised jobs
from executing while allowing other jobs to keep running.

The implementation leverages GitHub Actions context values to determine
what job to validate, resulting in an identical step in all jobs. This
makes it a good proof of concept for a ghasum action that can be used in
any job.

This commit is a followup to <2effc08890ff0aaa80d46a390489693b3c3deddc>.
@ericcornelissen ericcornelissen added ci/cd Relates to ci/cd test Relates to testing labels Mar 20, 2024
@ericcornelissen ericcornelissen merged commit 8dc7b4f into main Mar 20, 2024
10 checks passed
@ericcornelissen ericcornelissen deleted the ci-ghasum-verify-jobs branch March 20, 2024 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd Relates to ci/cd test Relates to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant