#314 Log errors when user is missing required role
Merged 4 years ago by lsedlar. Opened 4 years ago by lsedlar.
lsedlar/odcs log-forbidden  into  master

file modified
+3 -1
@@ -274,7 +274,9 @@ 

              in_users = flask.g.user.username in users

              if in_groups or in_users:

                  return f(*args, **kwargs)

-             raise Forbidden('User %s is not in role %s.' % (flask.g.user.username, role))

+             msg = "User %s is not in role %s." % (flask.g.user.username, role)

+             log.error(msg)

+             raise Forbidden(msg)

          return wrapped

      return wrapper

  

This should simplify adding new users when they can not easily get the returned error message.

rebased onto b70c302

4 years ago

Pull-Request has been merged by lsedlar

4 years ago
Metadata