#1580 cli: verify user in block-notification command
Merged 2 years ago by mikem. Opened 2 years ago by julian8628.
julian8628/koji fix-block-noti  into  master

file modified
+6 -2
@@ -7529,9 +7529,13 @@ 

          parser.error("--user requires admin permission")

  

      if options.user:

-         user_id = session.getUser(options.user)['id']

+         user_id = session.getUser(options.user, strict=True)['id']

      else:

-         user_id = session.getLoggedInUser()['id']

+         logged_in_user = session.getLoggedInUser()

+         if logged_in_user:

+             user_id = logged_in_user['id']

+         else:

+             parser.error("Please login with authentication or specify --user")

  

      if options.package:

          package_id = session.getPackageID(options.package)

Metadata Update from @julian8628:
- Pull-request tagged with: testing-ready

2 years ago

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done

2 years ago

:thumbsup:
worksforme

Commit ecb9829 fixes this pull-request

Pull-Request has been merged by mikem

2 years ago