-
-
Notifications
You must be signed in to change notification settings - Fork 375
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
Extend Gated Feature with an Allow/Block List #336
Comments
#1969 created a dedicated file to handle new cases in it :) |
Do you already have some ideas in mind on how this should be implemented? I would like to work on this, as at least for me, it gets annoying very fast 🙈 A repo setting to configure a list of authors that can be checked in https://github.com/woodpecker-ci/woodpecker/blob/main/server/pipeline/gated.go#L20 |
well there are ideas like:
so basically there are rules and you could have a allow- or a deny- block-list |
Would it make sense to integrate open policy agent for stuff like this? We could allow admins to write the own policies. We could also make use of it in e.g. extensions as proposed in #915 |
opa is a great tool, but for WP it feels to me it would add to much complexity. |
extend feature added (back) in #51
ref: https://codeberg.org/Codeberg/Community/issues/428#issuecomment-195258
ref: #193
The text was updated successfully, but these errors were encountered: