Merge pull request #305 from aoen/dd_fix_kerb_auth

Fix Kerberos Request Module

1 files changed, 11 insertions, 15 deletions

diff --git a/jenkinsapi/utils/krb_requester.py b/jenkinsapi/utils/krb_requester.py
index a9c4e7a..17319a4 100644
--- a/jenkinsapi/utils/krb_requester.py
+++ b/jenkinsapi/utils/krb_requester.py
@@ -14,7 +14,8 @@ class KrbRequester(Requester):
 
     def __init__(self, ssl_verify=None, baseurl=None, mutual_auth=OPTIONAL):
         """
-        :param ssl_verify: flag indicating if server certificate in HTTPS requests should be verified
+        :param ssl_verify: flag indicating if server certificate in HTTPS requests should be
+                           verified
         :param baseurl: Jenkins' base URL
         :param mutual_auth: type of mutual authentication, use one of REQUIRED, OPTIONAL or DISABLED
                             from requests_kerberos package
@@ -27,17 +28,12 @@ class KrbRequester(Requester):
         super(KrbRequester, self).__init__(**args)
         self.mutual_auth = mutual_auth
 
-    def get_request_dict(self, url, params, data, headers):
-        req_dict = Requester(
-            self,
-            url=url,
-            params=params,
-            data=data,
-            headers=headers
-        )
-        if self.mutual_auth:
-            auth = HTTPKerberosAuth(self.mutual_auth)
-        else:
-            auth = HTTPKerberosAuth()
-        req_dict['auth'] = auth
-        return req_dict
+    def get_request_dict(self, params=None, data=None, files=None, headers=None):
+      req_dict = super(KrbRequester, self).get_request_dict(params=params, data=data, files=files,
+                                                            headers=headers)
+      if self.mutual_auth:
+         auth = HTTPKerberosAuth(self.mutual_auth)
+      else:
+         auth = HTTPKerberosAuth()
+      req_dict['auth'] = auth
+      return req_dict