mailbox-org-api 2.4__tar.gz → 2.5__tar.gz

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mailbox-org-api
-Version: 2.4
+Version: 2.5
 Summary: A library to access the mailbox Business API.
 Author: Hendrik Schlange
 Author-email: Hendrik Schlange <mail@heshsum.de>

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/mailbox_org_api/APIClient.py

@@ -24,7 +24,7 @@ class APIClient:
     Object for API Client
     """
 
-    def __init__(self, debug_output=False, max_retries=5):
+    def __init__(self, debug_output=False, max_retries=5, request_timeout: int = 30):
         # URL of the API
         self.url = "https://api.mailbox.org/v1/"
 
@@ -60,6 +60,8 @@ class APIClient:
         adapter = HTTPAdapter(max_retries=retry_strategy)
         self.session.mount('https://', adapter)
 
+        self.request_timeout = request_timeout
+
     # Increment the request ID
     def get_jsonrpc_id(self):
         """Method to create the JSON RPC request ID. """
@@ -90,20 +92,26 @@ class APIClient:
             # Check if any parameter is 'password' or 'pass'.
             # Check explicit for these strings as not to interfere with possible other
             # parameters containing a substring.
+            sensitive_keys = ['password', 'pass', 'password_hash', 'new_account_password', 'token',
+                              'HPLS-AUTH', 'auth_id', 'telephone_password']
             for param in print_request['params']:
-                if param == 'password' or param == 'pass':
-                    # If True, replace the logged value with 'xxx'
+                if param in sensitive_keys:
                     print_request['params'][param] = 'xxx'
 
             # Print the clean version of the request
             print('API full request:\t', print_request)
 
-        api_response = self.session.post(self.url, data=json.dumps(request))
         try:
-            api_response = api_response.json()
+            response = self.session.post(self.url, json=request, timeout=self.request_timeout)
+            response.raise_for_status()
         except RequestException as error:
-            print('Non-JSON response received.\nFull response:\t', api_response,
-                  '\nError:', {error})
+            raise APIError(message=f"HTTP request failed: {error}", code=-32000) from error
+
+        try:
+            api_response = response.json()
+        except (ValueError, requests.exceptions.JSONDecodeError) as error:
+            print(f'API Full response: {response.content}')
+            raise APIError(message="Non-JSON response received from API", code=-32700) from error
         if self.debug_output:
             print('API full response:\t', api_response)
 
@@ -584,7 +592,7 @@ class APIClient:
         if 'password' in kwargs and 'password_hash' in kwargs:
             raise KeyError('''Simultaneous usage of 'password' and 'password_hash' not allowed.''')
 
-        if 'additional_mail_quota' in kwargs or 'additional_cloud_quota' in kwargs and 'plan' not in kwargs:
+        if ('additional_mail_quota' in kwargs or 'additional_cloud_quota' in kwargs) and 'plan' not in kwargs:
             raise KeyError('''If setting additional quota, 'plan' must be given.''')
 
         # Check for each argument in kwargs if it is a valid function parameter.
@@ -1111,16 +1119,13 @@ class APIClient:
                                 {'delete_mail_accounts_and_domains': delete_mail_accounts_and_domains})
 
 
-def validate_params(allowed: dict, actual: dict) -> bool | None:
+def validate_params(allowed: dict, actual: dict) -> bool:
     for arg in actual:
         if arg not in allowed:
             raise ValueError(f'Parameter {arg} not a valid parameter.')
-
         if not isinstance(actual[arg], allowed[arg]):
             raise TypeError(f'Attribute {arg} must be of type {str(allowed[arg])}. {str(type(actual[arg]))} given')
-        else:
-            return True
-    return None
+    return True
 
 
 def bool2str(state: bool) -> str:

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/mailbox_org_api.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mailbox-org-api
-Version: 2.4
+Version: 2.5
 Summary: A library to access the mailbox Business API.
 Author: Hendrik Schlange
 Author-email: Hendrik Schlange <mail@heshsum.de>

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mailbox-org-api"
-version = "2.4"
+version = "2.5"
 authors = [
   { name="Hendrik Schlange", email="mail@heshsum.de" },
 ]

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/setup.py

@@ -3,7 +3,7 @@ from setuptools import find_packages, setup
 setup(
     name='mailbox_org_api',
     packages=find_packages(),
-    version='2.4',
+    version='2.5',
     description='A library to access the mailbox Business API',
     author='Hendrik Schlange',
     install_requires=['requests'],

{mailbox_org_api-2.4 → mailbox_org_api-2.5}/tests/TestAPIClient.py

@@ -379,6 +379,9 @@ class TestAPIClient:
         api = APIClient.APIClient()
         api.auth(api_test_user, api_test_pass)
         mail = test_id + '@' + domain
+
+        # Ensure that the plan supports capabilities
+        api.mail_set_plan(mail, 'standard')
         for i in capabilities:
             api.mail_capabilities_set(mail, [i])
             # The API returns a list of capabilities
@@ -406,7 +409,7 @@ class TestAPIClient:
         api.auth(api_test_user, api_test_pass)
         mails = api.mail_list(domain)
         mail = mails[0]['mail']
-        plans = ['premium', 'light', 'standard']
+        plans = ['premium', 'standard']
         for plan in plans:
             api.mail_set_plan(mail, plan)
             assert str(api.mail_get(mail)['plan']).lower() == plan
@@ -468,16 +471,17 @@ class TestAPIClient:
         mail = test_id + '@' + domain
         additional_mail_quota = 23
         api.mail_set_additional_mail_quota(mail, additional_mail_quota)
-        assert additional_mail_quota == api.mail_get(mail)['additional_mail_quota']
+        assert int(api.mail_get(mail)['additional_mail_quota']) == additional_mail_quota
         api.deauth()
 
+    @pytest.mark.depends(name="test_mail_add")
     def test_mail_set_additional_cloud_quota(self):
         api = APIClient.APIClient()
         api.auth(api_test_user, api_test_pass)
         mail = test_id + '@' + domain
         additional_cloud_quota = 42
         api.mail_set_additional_cloud_quota(mail, additional_cloud_quota)
-        assert additional_cloud_quota == api.mail_get(mail)['additional_cloud_quota']
+        assert int(api.mail_get(mail)['additional_cloud_quota']) == additional_cloud_quota
         api.deauth()
 
     @pytest.mark.depends(name="test_mail_add")