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

Push temporary Docker images to GitHub Packages #341

Merged
merged 4 commits into from
Aug 25, 2023

Conversation

Witiko
Copy link
Owner

@Witiko Witiko commented Aug 25, 2023

GitHub security policy disallows sharing secrets with forks in public repositories. However, we need pull requests to push temporary Docker images to a registry, which requires access to secrets unless we use GitHub Packages, which only require the GITHUB_TOKEN secret that is shared with workflows triggered by pull requests from public repositories.

This pull request causes temporary Docker images to be pushed to GitHub Packages.

GitHub security policy disallows sharing secrets with forks in public
repositories [1]. However, we need pull requests to push temporary
Docker images to a registry, which requires access to secrets unless we
use GitHub Packages, which only require the `GITHUB_TOKEN` secret [2]
that is shared with workflows triggered by pull requests from public
repositories.

This commit causes temporary Docker images to be pushed to GitHub
Packages.

 [1]: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
 [2]: https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry
@Witiko Witiko added bug continuous integration Related to the continuous integration labels Aug 25, 2023
@Witiko Witiko added this to the 3.0.0 milestone Aug 25, 2023
@Witiko Witiko marked this pull request as ready for review August 25, 2023 12:15
@Witiko Witiko merged commit f0dd8fd into main Aug 25, 2023
12 checks passed
@Witiko Witiko deleted the feature/store-temporary-docker-images-in-github-packages branch August 25, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug continuous integration Related to the continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant