Merge pull request #76 from Adyen/develop

Release 2.1.0

Author: AlexandrosMor

Adyen/client.py

@@ -71,7 +71,7 @@ def __init__(self, username=None, password=None, xapikey=None,
                  store_payout_username=None, store_payout_password=None,
                  platform="test", merchant_account=None,
                  merchant_specific_url=None, skin_code=None,
-                 hmac=None, app_name=None,
+                 hmac=None, app_name="",
                  http_force=None, live_endpoint_prefix=None):
         self.username = username
         self.password = password
@@ -222,40 +222,46 @@ def call_api(self, request_data, service, action, idempotency=False,
 
         # username at self object has highest priority. fallback to root module
         # and ensure that it is set.
+        xapikey = None
         if self.xapikey:
             xapikey = self.xapikey
         elif 'xapikey' in kwargs:
             xapikey = kwargs.pop("xapikey")
 
+        username = None
         if self.username:
             username = self.username
         elif 'username' in kwargs:
             username = kwargs.pop("username")
-        elif service == "Payout":
+        if service == "Payout":
             if any(substring in action for substring in
                    ["store", "submit"]):
                 username = self._store_payout_username(**kwargs)
             else:
                 username = self._review_payout_username(**kwargs)
-        if not username:
+
+        if not username and not xapikey:
             errorstring = """Please set your webservice username.
               You can do this by running
               'Adyen.username = 'Your username'"""
             raise AdyenInvalidRequestError(errorstring)
             # password at self object has highest priority.
             # fallback to root module
             # and ensure that it is set.
-        if self.password:
+
+        password = None
+        if self.password and not xapikey:
             password = self.password
         elif 'password' in kwargs:
             password = kwargs.pop("password")
-        elif service == "Payout":
+        if service == "Payout":
             if any(substring in action for substring in
                    ["store", "submit"]):
                 password = self._store_payout_pass(**kwargs)
             else:
                 password = self._review_payout_pass(**kwargs)
-        if not password:
+
+        if not password and not xapikey:
             errorstring = """Please set your webservice password.
               You can do this by running
               'Adyen.password = 'Your password'"""

Adyen/httpclient.py

@@ -60,6 +60,7 @@ def _pycurl_post(self,
                      data=None,
                      username="",
                      password="",
+                     xapikey="",
                      headers={},
                      timeout=30):
         """This function will POST to the url endpoint using pycurl. returning
@@ -77,6 +78,8 @@ def _pycurl_post(self,
                 as part of password.
             password (str, optional): Password for basic auth. Must be included
                 as part of username.
+            xapikey (str, optional):    Adyen API key.  Will be used for auth
+                                        if username and password are absent.
             headers (dict, optional): Key/Value pairs of headers to include
             timeout (int, optional): Default 30. Timeout for the request.
 
@@ -102,6 +105,11 @@ def _pycurl_post(self,
         # request can be identified as coming from the Adyen Python library.
         headers['User-Agent'] = self.user_agent
 
+        if username and password:
+            curl.setopt(curl.USERPWD, '%s:%s' % (username, password))
+        elif xapikey:
+            headers["X-API-KEY"] = xapikey
+
         # Convert the header dict to formatted array as pycurl needs.
         if sys.version_info[0] >= 3:
             header_list = ["%s:%s" % (k, v) for k, v in headers.items()]
@@ -120,9 +128,6 @@ def _pycurl_post(self,
         raw_request = json_lib.dumps(json) if json else urlencode(data)
         curl.setopt(curl.POSTFIELDS, raw_request)
 
-        if username and password:
-            curl.setopt(curl.USERPWD, '%s:%s' % (username, password))
-
         curl.setopt(curl.TIMEOUT, timeout)
         curl.perform()
 
@@ -143,7 +148,7 @@ def _requests_post(self, url,
                        username="",
                        password="",
                        xapikey="",
-                       headers=None,
+                       headers={},
                        timeout=30):
         """This function will POST to the url endpoint using requests.
         Returning an AdyenResult object on 200 HTTP response.
@@ -160,6 +165,8 @@ def _requests_post(self, url,
                 as part of password.
             password (str, optional): Password for basic auth. Must be included
                 as part of username.
+            xapikey (str, optional):    Adyen API key.  Will be used for auth
+                                        if username and password are absent.
             headers (dict, optional): Key/Value pairs of headers to include
             timeout (int, optional): Default 30. Timeout for the request.
 
@@ -169,8 +176,6 @@ def _requests_post(self, url,
             int:    HTTP status code, eg 200,404,401
             dict:   Key/Value pairs of the headers received.
         """
-        if headers is None:
-            headers = {}
 
         # Adding basic auth if username and password provided.
         auth = None
@@ -194,11 +199,12 @@ def _requests_post(self, url,
         return request.text, message, request.status_code, request.headers
 
     def _urllib_post(self, url,
-                     json="",
-                     data="",
+                     json=None,
+                     data=None,
                      username="",
                      password="",
-                     headers=None,
+                     xapikey="",
+                     headers={},
                      timeout=30):
 
         """This function will POST to the url endpoint using urllib2. returning
@@ -216,6 +222,8 @@ def _urllib_post(self, url,
                                         uncluded as part of password.
             password (str, optional):   Password for basic auth. Must be
                                         included as part of username.
+            xapikey (str, optional):    Adyen API key.  Will be used for auth
+                                        if username and password are absent.
             headers (dict, optional):   Key/Value pairs of headers to include
             timeout (int, optional): Default 30. Timeout for the request.
 
@@ -225,8 +233,6 @@ def _urllib_post(self, url,
             int:    HTTP status code, eg 200,404,401
             dict:   Key/Value pairs of the headers received.
         """
-        if headers is None:
-            headers = {}
 
         # Store regular dict to return later:
         raw_store = json
@@ -258,6 +264,8 @@ def _urllib_post(self, url,
                     replace('\n', '')
             url_request.add_header("Authorization",
                                    "Basic %s" % basic_authstring)
+        elif xapikey:
+            headers["X-API-KEY"] = xapikey
 
         # Adding the headers to the request.
         for key, value in headers.items():
@@ -302,6 +310,8 @@ def request(self, url,
                                         uncluded as part of password.
             password (str, optional):   Password for basic auth. Must be
                                         included as part of username.
+            xapikey (str, optional):    Adyen API key.  Will be used for auth
+                                        if username and password are absent.
             headers (dict, optional):   Key/Value pairs of headers to include
         Returns:
             str:    Raw request placed

Adyen/settings.py

@@ -4,10 +4,10 @@
 ENDPOINT_CHECKOUT_TEST = "https://checkout-test.adyen.com"
 ENDPOINT_CHECKOUT_LIVE_SUFFIX = "https://{}-checkout-live" \
                                 ".adyenpayments.com/checkout"
-API_CHECKOUT_VERSION = "v41"
+API_CHECKOUT_VERSION = "v49"
 API_CHECKOUT_UTILITY_VERSION = "v1"
 API_RECURRING_VERSION = "v25"
-API_PAYMENT_VERSION = "v40"
+API_PAYMENT_VERSION = "v49"
 API_PAYOUT_VERSION = "v30"
-LIB_VERSION = "2.0.0"
+LIB_VERSION = "2.1.0"
 LIB_NAME = "adyen-python-api-library"

setup.py

@@ -3,7 +3,7 @@
 setup(
     name='Adyen',
     packages=['Adyen'],
-    version='2.0.0',
+    version='2.1.0',
     maintainer='Adyen',
     maintainer_email='[email protected]',
     description='Adyen Python Api',

test/BaseTest.py

@@ -1,4 +1,8 @@
-import mock
+
+try:
+    import mock
+except Exception:
+    from unittest import mock
 import json
 from Adyen import httpclient
 

test/DetermineEndpointTest.py

@@ -17,14 +17,14 @@ def test_checkout_api_url_custom(self):
         url = self.adyen.client._determine_checkout_url("live", "payments")
         self.client.live_endpoint_prefix = "1797a841fbb37ca7-AdyenDemo"
         self.assertEqual(url, "https://1797a841fbb37ca7-AdyenDemo-checkout-"
-                              "live.adyenpayments.com/checkout/v41/payments")
+                              "live.adyenpayments.com/checkout/v49/payments")
 
     def test_checkout_api_url(self):
         self.client.live_endpoint_prefix = None
         url = self.adyen.client._determine_checkout_url("test",
                                                         "paymentsDetails")
         self.assertEqual(url, "https://checkout-test.adyen.com"
-                              "/v41/payments/details")
+                              "/v49/payments/details")
 
     def test_payments_invalid_platform(self):