diff options
author | Ivan Kanakarakis <ivan.kanak@gmail.com> | 2020-09-10 02:28:38 +0300 |
---|---|---|
committer | Ivan Kanakarakis <ivan.kanak@gmail.com> | 2020-09-11 02:11:12 +0300 |
commit | bc96c3856dfd6ffbd27e0f59acacfef2b71e4edd (patch) | |
tree | 543f78595d25fd3127199e26917df9b3c67c156b /src/saml2/sigver.py | |
parent | 7b1b52e03f06310bc23f688fe3f373881950a9f7 (diff) | |
download | pysaml2-assert_deletion.tar.gz |
Replace assert with proper checksassert_deletion
Signed-off-by: Ivan Kanakarakis <ivan.kanak@gmail.com>
Diffstat (limited to 'src/saml2/sigver.py')
-rw-r--r-- | src/saml2/sigver.py | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/saml2/sigver.py b/src/saml2/sigver.py index 3cf7c215..0d960d21 100644 --- a/src/saml2/sigver.py +++ b/src/saml2/sigver.py @@ -679,7 +679,8 @@ class CryptoBackendXmlSec1(CryptoBackend): def __init__(self, xmlsec_binary, delete_tmpfiles=True, **kwargs): CryptoBackend.__init__(self, **kwargs) - assert (isinstance(xmlsec_binary, six.string_types)) + if not isinstance(xmlsec_binary, six.string_types): + raise ValueError("xmlsec_binary should be of type string") self.xmlsec = xmlsec_binary self.delete_tmpfiles = delete_tmpfiles try: @@ -1237,11 +1238,12 @@ class SecurityContext(object): sec_backend=None, delete_tmpfiles=True): + if not isinstance(crypto, CryptoBackend): + raise ValueError("crypto should be of type CryptoBackend") self.crypto = crypto - assert (isinstance(self.crypto, CryptoBackend)) - if sec_backend: - assert (isinstance(sec_backend, RSACrypto)) + if sec_backend and not isinstance(sec_backend, RSACrypto): + raise ValueError("sec_backend should be of type RSACrypto") self.sec_backend = sec_backend # Your private key for signing |