From d90fb86318f21e1f893a17af56700010bca9e418 Mon Sep 17 00:00:00 2001 From: Tomas Kopecek Date: Sep 08 2021 15:21:37 +0000 Subject: PR#2999: set-task-priority fix permission error Merges #2999 https://pagure.io/koji/pull-request/2999 Fixes: #2969 https://pagure.io/koji/issue/2969 set-task-priority prints confusing error --- diff --git a/cli/koji_cli/commands.py b/cli/koji_cli/commands.py index 0f37fa7..bfbaeee 100644 --- a/cli/koji_cli/commands.py +++ b/cli/koji_cli/commands.py @@ -6618,6 +6618,10 @@ def handle_set_task_priority(goptions, session, args): activate_session(session, goptions) + if not session.hasPerm('admin'): + logged_user = session.getLoggedInUser() + error("admin permission required (logged in as %s)" % logged_user['name']) + for task_id in tasks: try: session.setTaskPriority(task_id, options.priority, options.recurse)