Skip to content

Commit 7f37b47

Browse files
committed
Move clean summary function to AdvisoryData class
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
1 parent 8a910de commit 7f37b47

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

vulnerabilities/import_runner.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,6 @@ def process_advisories(
9191
# https://nvd.nist.gov/vuln/detail/CVE-2013-4314
9292
# https://github.com/cms-dev/cms/issues/888#issuecomment-516977572
9393
try:
94-
data.summary = data.summary.replace("\x00", "\uFFFD")
9594
obj, created = Advisory.objects.get_or_create(
9695
aliases=data.aliases,
9796
summary=data.summary,

vulnerabilities/importer.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -249,9 +249,14 @@ class AdvisoryData:
249249
date_published: Optional[datetime.datetime] = None
250250
weaknesses: List[int] = dataclasses.field(default_factory=list)
251251

252+
252253
def __post_init__(self):
253254
if self.date_published and not self.date_published.tzinfo:
254255
logger.warning(f"AdvisoryData with no tzinfo: {self!r}")
256+
self.clean_summary()
257+
258+
def clean_summary(self):
259+
self.summary = self.summary.replace("\x00", "\uFFFD")
255260

256261
def to_dict(self):
257262
return {

0 commit comments

Comments
 (0)