#490 getUserPerms should throw GenericError when no user found
Merged by mikem. Opened by julian8628.
julian8628/koji get-user-perm  into  master

Download 490.patch

to fix issue#486

When specified userID is not found, GenericError will be thrown instead of [] result

GenericError: No such user: xxxx

If user exists and no perm for this user, the result will keep [] as usual time.

This change will also make getUserPerms accepts username as argument. It's not the same as the old behavior.

1 new commit added

  • allow getUserPerm accepting None arg

also submit a commit#f36d487 to make None as the default value of this API's argument -- userID.
If no userID provided when invoking. it will get current login user's perms as default.

Can you expand docstring on this new behaviour?

Can you expand docstring on this new behaviour?

Updated :smoking:

1 new commit added

  • explain new behavior in docstring of getUserPerms

:thumbsup:

Commit 801e1662 fixes this pull-request

Pull-Request has been merged by mikem@redhat.com

Metadata