Merge pull request #120 from intezer/feature/add-invalid-url-error

feat(url analysis): Add invalid url error TKT-1270

CHANGES

@@ -1,3 +1,8 @@
+1.19.6
+------
+- Add InvalidUrlError
+- Add details to analysis server error
+
 1.19.5
 ------
 - Latest URL analysis fallback.

intezer_sdk/__init__.py

@@ -1 +1 @@
-__version__ = '1.19.5'
+__version__ = '1.19.6'

intezer_sdk/_api.py

@@ -741,11 +741,15 @@ def _assert_analysis_response_status_code(response: Response):
         elif response.status_code == HTTPStatus.BAD_REQUEST:
             data = response.json()
             error = data.get('error', '')
+            details = data.get('details', '')
             result = data.get('result', {})
             if result.get('is_url_offline'):
                 raise errors.UrlOfflineError(response)
 
-            raise errors.ServerError(f'Server returned bad request error: {error}', response)
+            if error == 'Invalid url':
+                raise errors.InvalidUrlError(response)
+
+            raise errors.ServerError(f'Server returned bad request error: {error}, details: {details}', response)
         elif response.status_code != HTTPStatus.CREATED:
             raise errors.ServerError(f'Error in response status code:{response.status_code}', response)
 

intezer_sdk/errors.py

@@ -56,10 +56,12 @@ class HashDoesNotExistError(ServerError):
     def __init__(self, response: requests.Response):
         super().__init__('Hash was not found', response)
 
+
 class FileTooLargeError(ServerError):
     def __init__(self, response: requests.Response):
         super().__init__('File is too large', response)
 
+
 class ReportDoesNotExistError(IntezerError):
     def __init__(self):
         super().__init__('Report was not found')
@@ -156,10 +158,17 @@ class AlertNotFoundError(AlertError):
     def __init__(self, alert_id: str):
         super().__init__(f'The given alert does not exist - {alert_id}')
 
+
 class InvalidAlertArgumentError(AlertError):
     def __init__(self, message: str):
         super().__init__(message)
 
+
 class UrlOfflineError(ServerError):
     def __init__(self, response: requests.Response):
         super().__init__('Url is offline', response)
+
+
+class InvalidUrlError(ServerError):
+    def __init__(self, response: requests.Response):
+        super().__init__('Invalid url', response)