simplify returns

Author: jschlyter

src/cryptojwt/jwk/asym.py

@@ -58,10 +58,7 @@ def has_private_key(self):
 
         :return: True/False
         """
-        if self.priv_key:
-            return True
-        else:
-            return False
+        return bool(self.priv_key)
 
     def public_key(self):
         """

src/cryptojwt/jwk/ec.py

@@ -249,10 +249,8 @@ def __eq__(self, other):
             if other.private_key():
                 if cmp_keys(self.priv_key, other.priv_key, ec.EllipticCurvePrivateKey):
                     return True
-            elif self.private_key():
-                return False
             else:
-                return True
+                return not self.private_key()
 
         return False
 

src/cryptojwt/jwk/okp.py

@@ -278,10 +278,8 @@ def __eq__(self, other):
             if other.private_key():
                 if cmp_keys(self.priv_key, other.priv_key, _private_cls):
                     return True
-            elif self.private_key():
-                return False
             else:
-                return True
+                return not self.private_key()
 
         return False
 

src/cryptojwt/jwk/rsa.py

@@ -117,13 +117,7 @@ def rsa_eq(key1, key2):
     :param key2:
     :return:
     """
-    pn1 = key1.public_numbers()
-    pn2 = key2.public_numbers()
-    # Check if two RSA keys are in fact the same
-    if pn1 == pn2:
-        return True
-    else:
-        return False
+    return key1.public_numbers() == key2.public_numbers()
 
 
 def x509_rsa_load(txt):

src/cryptojwt/key_jar.py

@@ -256,10 +256,7 @@ def get_issuer_keys(self, issuer_id):
     @deprecated_alias(issuer="issuer_id", owner="issuer_id")
     def __contains__(self, issuer_id):
         _iss = self._get_issuer(issuer_id)
-        if _iss is None:
-            return False
-        else:
-            return True
+        return not _iss is None
 
     @deprecated_alias(issuer="issuer_id", owner="issuer_id")
     def __getitem__(self, issuer_id=""):

src/cryptojwt/simple_jwt.py

@@ -111,15 +111,9 @@ def verify_header(self, key, val):
         """
 
         if isinstance(val, list):
-            if self.headers[key] in val:
-                return True
-            else:
-                return False
+            return self.headers[key] in val
         else:
-            if self.headers[key] == val:
-                return True
-            else:
-                return False
+            return self.headers[key] == val
 
     def verify_headers(self, check_presence=True, **kwargs):
         """