hub: more verbose assertPerm error

Fixes: https://pagure.io/koji/issue/2429
2020-08-25 13:29:32 +02:00 · 2020-08-25 13:29:32 +02:00 · b4cfca052b
commit b4cfca052b
parent c5c1c86629
1 changed files with 6 additions and 1 deletions
--- a/koji/auth.py
+++ b/koji/auth.py
@ -501,7 +501,12 @@ class Session(object):

    def assertPerm(self, name):
        if not self.hasPerm(name) and not self.hasPerm('admin'):
-            raise koji.ActionNotAllowed("%s permission required" % name)
+            msg = "%s permission required" % name
+            if self.logged_in:
+                msg += ' (logged in as %s)' % self.user_data['name']
+            else:
+                msg += ' (user not logged in)'
+            raise koji.ActionNotAllowed(msg)

    def assertLogin(self):
        if not self.logged_in: