Clean up self.verified (Closes: SecurityInnovation#457)

dkg · Jul 23, 2023 · 8682b1f · 8682b1f
1 parent 30a7571
commit 8682b1f
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/pgpy/pgp.py b/pgpy/pgp.py
@@ -2385,13 +2385,11 @@ def _do_self_signatures_verification(self):
             raise
 
     @property
-    def self_verified(self):
-        warnings.warn("TODO: Self-sigs verification is not yet working because self-sigs are not parsed!!!")
-        return SecurityIssues.OK
-
+    def self_verified(self) -> SecurityIssues:
         if self._self_verified is None:
             self._do_self_signatures_verification()
-
+        if self._self_verified is None:
+            raise ValueError("PGPKey._do_self_signatures_verification() should have set self._self_verified")
         return self._self_verified
 
     def check_primitives(self):