Merge pull request #131 from intezer/feat/add-skipped-error

feature(errors) - add AnalysisSkippedByRuleError TKT-2287
intezer · Jan 17, 2024 · 60b36b0 · 60b36b0
2 parents 94ac54a + 1c63b07
commit 60b36b0
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/CHANGES b/CHANGES
@@ -1,3 +1,7 @@
+1.19.12
+_______
+- Raise AnalysisSkippedByRuleError when analysis is skipped by rule on server
+
 1.19.11
 _______
 - Add AnalysisSkippedByRuleError

diff --git a/intezer_sdk/__init__.py b/intezer_sdk/__init__.py
@@ -1 +1 @@
-__version__ = '1.19.11'
+__version__ = '1.19.12'
diff --git a/intezer_sdk/api.py b/intezer_sdk/api.py
@@ -43,6 +43,10 @@ def raise_for_status(response: requests.Response,
     elif 400 <= response.status_code < 500:
         if response.status_code == HTTPStatus.UNAUTHORIZED:
             raise errors.InvalidApiKeyError(response)
+        elif response.status_code == HTTPStatus.CONFLICT:
+            is_skipped_by_rule = response.json().get('result', {}).get('is_skipped_by_rule')
+            if is_skipped_by_rule:
+                raise errors.AnalysisSkippedByRuleError(response)
         elif response.status_code == HTTPStatus.FORBIDDEN:
             try:
                 error_message = response.json()['error']
@@ -671,6 +675,10 @@ def _assert_analysis_response_status_code(response: Response):
         if response.status_code == HTTPStatus.NOT_FOUND:
             raise errors.HashDoesNotExistError(response)
         elif response.status_code == HTTPStatus.CONFLICT:
+            is_skipped_by_rule = response.json().get('result', {}).get('is_skipped_by_rule')
+            if is_skipped_by_rule:
+                raise errors.AnalysisSkippedByRuleError(response)
+
             running_analysis_id = response.json().get('result', {}).get('analysis_id')
             raise errors.AnalysisIsAlreadyRunningError(response, running_analysis_id)
         elif response.status_code == HTTPStatus.FORBIDDEN: