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

Feature: Add a Tag protection check to encourage stronger access control across releases #2476

Open
inferno-chromium opened this issue Nov 21, 2022 · 1 comment
Labels
kind/enhancement New feature or request priority/must-do Upcoming release

Comments

@inferno-chromium
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Right now, a write/maintain role contributor can delete existing releases and replace with releases pointing to older, vulnerable commits. Read why - https://medium.com/boostsecurity/slsa-dip-source-of-the-problem-a1dac46a976. Have your release tags protected - https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/configuring-tag-protection-rules

Describe the solution you'd like
Maybe a check that encourages adding atleast one tag protection rule on existing releases. Ideally a "*" would be nice, so only admins can have those perms to replace/delete releases.

Describe alternatives you've considered
None

Additional context
None

@inferno-chromium inferno-chromium added kind/enhancement New feature or request priority/must-do Upcoming release labels Nov 21, 2022
@varunsh-coder
Copy link
Contributor

I believe this was also mentioned in @mbarbero's blog post
https://mikael.barbero.tech/blog/post/eclipsefdn-scorecard-aug2022/

Also, release tags should be protected as well to prevent compromised contributors to change tags to some commit with malicious code. Note that tags protection is not currently checked by scorecard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request priority/must-do Upcoming release
Projects
Status: No status
Development

No branches or pull requests

2 participants