Merge pull request #57 from Evert-Arends/patch-1

Updated standards and fixed problems with python 3.8

Author: claudiosanches

woocommerce/api.py

@@ -5,14 +5,15 @@
 """
 
 __title__ = "woocommerce-api"
-__version__ = "2.1.1"
+__version__ = "2.1.2"
 __author__ = "Claudio Sanches @ Automattic"
 __license__ = "MIT"
 
 from requests import request
 from json import dumps as jsonencode
 from time import time
 from woocommerce.oauth import OAuth
+from requests.auth import HTTPBasicAuth
 
 try:
     from urllib.parse import urlencode
@@ -44,12 +45,12 @@ def __get_url(self, endpoint):
         api = "wc-api"
 
         if url.endswith("/") is False:
-            url = "%s/" % url
+            url = f"{url}/"
 
         if self.wp_api:
             api = "wp-json"
 
-        return "%s%s/%s/%s" % (url, api, self.version, endpoint)
+        return f"{url}{api}/{self.version}/{endpoint}"
 
     def __get_oauth_url(self, url, method, **kwargs):
         """ Generate oAuth1.0a URL """
@@ -71,20 +72,20 @@ def __request(self, method, endpoint, data, params=None, **kwargs):
         url = self.__get_url(endpoint)
         auth = None
         headers = {
-            "user-agent": "WooCommerce API Client-Python/%s" % __version__,
+            "user-agent": f'WooCommerce API {__version__}',
             "accept": "application/json"
         }
 
         if self.is_ssl is True and self.query_string_auth is False:
-            auth = (self.consumer_key, self.consumer_secret)
+            auth = HTTPBasicAuth(self.consumer_key, self.consumer_secret)
         elif self.is_ssl is True and self.query_string_auth is True:
             params.update({
                 "consumer_key": self.consumer_key,
                 "consumer_secret": self.consumer_secret
             })
         else:
             encoded_params = urlencode(params)
-            url = "%s?%s" % (url, encoded_params)
+            url = f"{url}?{encoded_params}"
             url = self.__get_oauth_url(url, method, **kwargs)
 
         if data is not None:
@@ -122,4 +123,3 @@ def delete(self, endpoint, **kwargs):
     def options(self, endpoint, **kwargs):
         """ OPTIONS requests """
         return self.__request("OPTIONS", endpoint, None, **kwargs)
-