Remove debug statements, improve doc, simplify code

Signed-off-by: Shivam Sandbhor <shivam.sandbhor@gmail.com>

Author: sbs2001

vulnerabilities/importers/apache_tomcat.py

@@ -111,8 +111,6 @@ def to_advisories(self, apache_tomcat_advisory_html):
                         type="maven", namespace="apache", name="tomcat", version=fixed_version
                     )
                 ]
-                if fixed_package in affected_packages:
-                    raise ValueError("Bing bong")
 
                 advisories.append(
                     Advisory(

vulnerabilities/importers/istio.py

@@ -31,7 +31,6 @@
 
 from vulnerabilities.data_source import Advisory, GitDataSource, Reference
 from vulnerabilities.package_managers import GitHubTagsAPI
-from vulnerabilities.helpers import contains_alpha
 
 is_release = re.compile(r"^[\d.]+$", re.IGNORECASE).match
 
@@ -49,8 +48,6 @@ def __enter__(self):
 
     def set_api(self):
         asyncio.run(self.version_api.load_api(["istio/istio"]))
-        # Only keep public releases
-        self.version_api.cache = list(filter(contains_alpha, self.version_api.cache))
 
     def updated_advisories(self) -> Set[Advisory]:
         files = self._added_files.union(self._updated_files)

vulnerabilities/importers/npm.py

@@ -130,7 +130,9 @@ def _versions_to_purls(package_name, versions):
 
 def normalize_ranges(version_range_string):
     """
-    Splits version range strings with "||" operator into separate ranges
+    - Splits version range strings with "||" operator into separate ranges.
+    - Removes spaces between range operator and range operands
+    - Normalizes 'x' ranges
     Example:
     >>> z = normalize_ranges(">=6.1.3 < 7.0.0 || >=7.0.3")
     >>> assert z == [">=6.1.3,<7.0.0", ">=7.0.3"]