aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorGeorgeFI2023-06-10 10:41:37 +0200
committerGeorgeFI2023-06-10 10:41:37 +0200
commitd17f17b096089b8ea858fdd59c2756880c1ec54f (patch)
tree48eb084ed70c9e8d8932ca36107ee377a146ca2d
parentd7692fdf295db769b407abfd537ea4ac21b086a9 (diff)
downloadsec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.tar.gz
sec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.tar.zst
sec-certs-d17f17b096089b8ea858fdd59c2756880c1ec54f.zip
refactor: Light refactor of fips returns statements
-rw-r--r--src/sec_certs/sample/fips.py9
1 files changed, 3 insertions, 6 deletions
diff --git a/src/sec_certs/sample/fips.py b/src/sec_certs/sample/fips.py
index d351c3ce..26510191 100644
--- a/src/sec_certs/sample/fips.py
+++ b/src/sec_certs/sample/fips.py
@@ -83,15 +83,13 @@ class FIPSHTMLParser:
def _build_vendor_dict(vendor_div: Tag) -> dict[str, Any]:
if not (link := vendor_div.find("a")):
return {"vendor_url": None, "vendor": list(vendor_div.find("div", "panel-body").children)[0].strip()}
- else:
- return {"vendor_url": link.get("href"), "vendor": link.text.strip()}
+ return {"vendor_url": link.get("href"), "vendor": link.text.strip()}
@staticmethod
def _build_related_files_dict(related_files_div: Tag) -> dict[str, Any]:
if cert_link := [x for x in related_files_div.find_all("a") if "Certificate" in x.text]:
return {"certificate_pdf_url": constants.FIPS_BASE_URL + cert_link[0].get("href")}
- else:
- return {"certificate_pdf_url": None}
+ return {"certificate_pdf_url": None}
@staticmethod
def _build_validation_history_dict(validation_history_div: Tag) -> dict[str, Any]:
@@ -433,8 +431,7 @@ class FIPSCertificate(
fips_certlike = self.keywords["fips_certlike"].get("Certlike", {})
matches = {re.search(r"#\s{0,1}\d{1,4}", x) for x in fips_certlike}
return {"".join([x for x in match.group() if x.isdigit()]) for match in matches if match}
- else:
- return set()
+ return set()
@dataclass(eq=True)
class Heuristics(BaseHeuristics, ComplexSerializableType):