#3686 Let "--principal=" works for users using multiple TGT's
Merged a year ago by tkopecek. Opened a year ago by tkopecek.
tkopecek/koji pr3683  into  master

file modified
+6 -5
@@ -757,11 +757,12 @@ 

          session.login()

      elif options.authtype == "kerberos" or options.authtype is None:

          try:

-             if getattr(options, 'keytab', None) and getattr(options, 'principal', None):

-                 session.gssapi_login(principal=options.principal, keytab=options.keytab,

-                                      proxyuser=runas)

-             else:

-                 session.gssapi_login(proxyuser=runas)

+             kwargs = {'proxyuser': runas}

+             if getattr(options, 'principal', None):

+                 kwargs['principal'] = options.principal

+                 if getattr(options, 'keytab', None):

+                     kwargs['keytab'] = options.keytab

+             session.gssapi_login(**kwargs)

          except socket.error as e:

              warn("Could not connect to Kerberos authentication service: %s" % e.args[1])

      if not noauth and not session.logged_in:

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

a year ago

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

a year ago

Commit b715e53 fixes this pull-request

Pull-Request has been merged by tkopecek

a year ago