From 3233dc760f12643bde4c20adca6ccf1eeb4a71e8 Mon Sep 17 00:00:00 2001 From: Mike McLean Date: Sep 15 2017 02:59:11 +0000 Subject: don't assume runas option in activate_session --- diff --git a/cli/koji_cli/lib.py b/cli/koji_cli/lib.py index 2f3aa8e..124780a 100644 --- a/cli/koji_cli/lib.py +++ b/cli/koji_cli/lib.py @@ -543,18 +543,18 @@ def activate_session(session, options): if isinstance(options, dict): options = optparse.Values(options) noauth = options.authtype == "noauth" or getattr(options, 'noauth', False) + runas = getattr(options, 'runas', None) if noauth: #skip authentication pass elif options.authtype == "ssl" or os.path.isfile(options.cert) and options.authtype is None: # authenticate using SSL client cert - session.ssl_login(options.cert, None, options.serverca, proxyuser=options.runas) + session.ssl_login(options.cert, None, options.serverca, proxyuser=runas) elif options.authtype == "password" or getattr(options, 'user', None) and options.authtype is None: # authenticate using user/password session.login() elif options.authtype == "kerberos" or has_krb_creds() and options.authtype is None: try: - runas = getattr(options, 'runas', None) if options.keytab and options.principal: session.krb_login(principal=options.principal, keytab=options.keytab, proxyuser=runas) else: