Skip to content

Commit c4da6e2

Browse files
authored
Merge pull request #144 from fosslight/exclude
Modify when exceeding the max. url length
2 parents 40ad99a + 4968e9a commit c4da6e2

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ py-tlsh
88
pytz
99
XlsxWriter
1010
PyYAML
11-
fosslight_util>=2.1.10
11+
fosslight_util>=2.1.13
1212
dependency-check

src/fosslight_binary/_binary.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,13 @@ def set_oss_items(self, new_oss_list, exclude=False, exclude_msg=""):
4949
# Append New input OSS
5050
self.oss_items.extend(new_oss_list)
5151

52-
def get_vulnerability_items(self):
53-
nvd_url = [urllib.parse.unquote(vul_item.nvd_url) for vul_item in self.vulnerability_items]
54-
return ", ".join(nvd_url).strip()
52+
def get_vulnerability_items(self, oss):
53+
nvd_url = set([urllib.parse.unquote(vul_item.nvd_url) for vul_item in self.vulnerability_items])
54+
nvd_url = ", ".join(nvd_url).strip()
55+
56+
if nvd_url and len(nvd_url) > MAX_EXCEL_URL_LENGTH:
57+
oss.comment += f"\nExceeded the maximum vulnerability URL length of {MAX_EXCEL_URL_LENGTH} characters."
58+
return nvd_url
5559

5660
def get_print_binary_only(self):
5761
return (self.source_name_or_path + "\t" + self.checksum + "\t" + self.tlsh)
@@ -62,12 +66,7 @@ def get_print_array(self):
6266
for oss in self.oss_items:
6367
lic = ",".join(oss.license)
6468
exclude = EXCLUDE_TRUE_VALUE if (self.exclude or oss.exclude) else ""
65-
nvd_url = self.get_vulnerability_items()
66-
if nvd_url and len(nvd_url) > MAX_EXCEL_URL_LENGTH:
67-
logger.info(f"self.source_name_or_path: {self.source_name_or_path}")
68-
logger.info(f"oss name: {oss.name}")
69-
logger.info(f"length_url:{len(nvd_url)}, nvd_url: {nvd_url}")
70-
nvd_url = nvd_url[:242] + '...(truncated)'
69+
nvd_url = self.get_vulnerability_items(oss)
7170
items.append([self.source_name_or_path, oss.name, oss.version,
7271
lic, oss.download_location, oss.homepage,
7372
oss.copyright, exclude, oss.comment,

0 commit comments

Comments
 (0)