From 39b3f92abc00f1b878c5c59e45f580c5c7291fba Mon Sep 17 00:00:00 2001 From: Ryan Lerch Date: Tue, 22 Jun 2021 19:32:58 +1000 Subject: [PATCH] distgit/pagure: remove the commit ACL for API keys in stg This was removed in prod in d0a8837 but left around in stg for testing purposes. However, it is suspected that this was causing users to not be able to push to forks in distgit, so removing. https://pagure.io/fedora-infrastructure/issue/10045 Signed-off-by: Ryan Lerch --- roles/distgit/pagure/templates/pagure.cfg | 9 --------- 1 file changed, 9 deletions(-) diff --git a/roles/distgit/pagure/templates/pagure.cfg b/roles/distgit/pagure/templates/pagure.cfg index d96e829bed..badbce45e4 100644 --- a/roles/distgit/pagure/templates/pagure.cfg +++ b/roles/distgit/pagure/templates/pagure.cfg @@ -233,9 +233,6 @@ ACLS = { 'commit_flag': 'Flag a commit', "update_watch_status": "Update the watch status on a project", "tag_project": "Allows adding git tags to a project", -{% if env == 'staging' %} - "commit": "Commit to a git repository via http(s)", -{% endif %} } REQUIRED_GROUPS = { @@ -266,9 +263,6 @@ USER_ACLS = [ 'pull_request_merge', 'modify_project', 'update_watch_status', -{% if env == 'staging' %} - 'commit', -{% endif %} ] CROSS_PROJECT_ACLS = [ @@ -279,9 +273,6 @@ CROSS_PROJECT_ACLS = [ 'pull_request_merge', 'pull_request_flag', 'update_watch_status', -{% if env == 'staging' %} - 'commit', -{% endif %} ] ADMIN_API_ACLS = [