From ecb9829c0750e0314702e89f36b86d473739f234 Mon Sep 17 00:00:00 2001
From: Mike McLean <mikem@redhat.com>
Date: Jul 30 2019 20:18:19 +0000
Subject: PR#1580: cli: verify user in block-notification command


Merges #1580
https://pagure.io/koji/pull-request/1580

Fixes: #1579
https://pagure.io/koji/issue/1579
block-notification CLI should raise GenericError exception for non existing user

---

diff --git a/cli/koji_cli/commands.py b/cli/koji_cli/commands.py
index e53a7f5..fb4ac9b 100644
--- a/cli/koji_cli/commands.py
+++ b/cli/koji_cli/commands.py
@@ -7529,9 +7529,13 @@ def handle_block_notification(goptions, session, args):
         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)