Fix issue where html_report_url did not have a hasattr() check (#13)

* Fix issue where html_report_url did not have a hasattr() check * Changed back to report.commit from report.commit_hash for backwards support
SocketDev · Oct 3, 2024 · ebc53a2 · ebc53a2
1 parent 920b663
commit ebc53a2
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/socketsync/__init__.py b/socketsync/__init__.py
@@ -2,7 +2,7 @@
 
 
 __author__ = 'socket.dev'
-__version__ = '1.0.12'
+__version__ = '1.0.14'
 __all__ = [
     "log",
     "__version__",

diff --git a/socketsync/classes.py b/socketsync/classes.py
@@ -57,7 +57,7 @@ def __init__(self, **kwargs):
                 self.pull_requests = [self.pull_request]
         if not hasattr(self, 'pull_requests') and hasattr(self, 'pull_request'):
             self.pull_requests = [self.pull_request]
-        if self.html_report_url is not None:
+        if hasattr(self, 'html_report_url') and self.html_report_url is not None:
             self.url = self.html_report_url
         if not hasattr(self, 'commit'):
             self.commit = self.commit_hash

diff --git a/socketsync/core.py b/socketsync/core.py
@@ -322,7 +322,7 @@ def create_issue_alerts(package: Package, alerts: list, packages: dict, report:
                 branch=report.branch,
                 report_id=report.id,
                 pr=pr,
-                commit=report.commit_hash,
+                commit=report.commit,
                 created_at=report.created_at,
                 pkg_type=package.type,
                 pkg_name=package.name,