Merge pull request #203 from GeekMasher/octokit-errors-improve

feat: Impove octokit error handling

Author: GeekMasher

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ghastoolkit"
-version = "0.12.1"
+version = "0.12.2"
 authors = [{ name = "GeekMasher" }]
 description = "GitHub Advanced Security Python Toolkit"
 readme = "README.md"

src/ghastoolkit/__init__.py

@@ -3,7 +3,7 @@
 __name__ = "ghastoolkit"
 __title__ = "GHAS Toolkit"
 
-__version__ = "0.12.1"
+__version__ = "0.12.2"
 
 __description__ = "GitHub Advanced Security Python Toolkit"
 __summary__ = """\

src/ghastoolkit/octokit/octokit.py

@@ -227,11 +227,15 @@ def get(
                     return error_handler(response.status_code, response_json)
 
                 # Default error handling
-                logger.error(response_json.get("message"))
-                logger.error(
-                    f"Documentation Link :: {response_json.get('documentation_url', '')}"
+                message = response_json.get("message", "No message provided")
+                docs = response_json.get(
+                    "documentation_url", "No documentation link provided"
                 )
-                raise Exception("REST Request failed :: non-expected server error")
+
+                logger.error(f"Error message from server :: {message}")
+                logger.error(f"Documentation Link :: {docs}")
+
+                raise Exception(f"REST Request failed :: {message}")
 
             if isinstance(response_json, dict):
                 return response_json