Add logger

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>

Author: TG1999

vulnerabilities/importers/redhat.py

@@ -80,19 +80,23 @@ def to_advisory(advisory_data):
     for rpm in advisory_data.get("affected_packages") or []:
         purl = rpm_to_purl(rpm)
         if purl:
-            affected_packages.append(
-                AffectedPackage(
-                    package=PackageURL(
-                        type=purl.type,
-                        name=purl.name,
-                        namespace=purl.namespace,
-                        qualifiers=purl.qualifiers,
-                        subpath=purl.subpath,
-                    ),
-                    affected_version_range=RpmVersionRange.from_native(f"= {purl.version}"),
-                    fixed_version=None,
+            try:
+                affected_version_range = RpmVersionRange.from_native(f"= {purl.version}")
+                affected_packages.append(
+                    AffectedPackage(
+                        package=PackageURL(
+                            type=purl.type,
+                            name=purl.name,
+                            namespace=purl.namespace,
+                            qualifiers=purl.qualifiers,
+                            subpath=purl.subpath,
+                        ),
+                        affected_version_range=affected_version_range,
+                        fixed_version=None,
+                    )
                 )
-            )
+            except Exception as e:
+                logger.error(f"Failed to parse version range {purl.version} for {purl}")
 
     references = []
     bugzilla = advisory_data.get("bugzilla")
@@ -196,9 +200,11 @@ def __new__(self, version, release=None, epoch=None):
         """
         if not isinstance(epoch, int):
             if epoch and epoch.strip():
-                raise ValueError("Invalid epoch: must be a number or empty.")
+                logger.error("Invalid epoch: must be a number or empty.")
+                return None
         if not version:
-            raise ValueError("Version is required: {}".format(repr(version)))
+            logger.error("Version is required: {}".format(repr(version)))
+            return None
 
         return super().__new__(EVR, epoch, version, release)
 
@@ -233,7 +239,8 @@ def from_name(rpm_string):
     if ":" not in v:
         return None, n, v, r, a
     e, v = v.split(":", 1)
-    e = int(e)
+    if e.isdigit():
+        e = int(e)
     return (e, n, v, r, a)
 
 
@@ -251,9 +258,15 @@ def rpm_to_purl(rpm_string):
     envra = from_name(rpm_string)
 
     if not envra:
+        logger.error(f"Invalid RPM name can't get envra: {rpm_string}")
         return None
     sepoch, sname, sversion, srel, sarch = envra
-    src_evr = EVR(sversion, srel, sepoch).to_string()
+
+    evr = EVR(sversion, srel, sepoch)
+    if not evr:
+        logger.error(f"Invalid RPM name can't get evr: {rpm_string}")
+        return None
+    src_evr = evr.to_string()
     src_qualifiers = {}
     if sarch:
         src_qualifiers["arch"] = sarch