Merge pull request #2 from dnmvisser/dv_pysaml-7.1-fix

Fix for pysaml 7.1
dnmvisser · May 31, 2022 · 5d4769c · 5d4769c
2 parents 04afa4c + f9fff6b
commit 5d4769c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -7,6 +7,10 @@ Currently implemented checks:
 - The validity of the TLS certificate of the Assertion Consumer URL
 - The validity of the SAML signing/encryption certificate(s)
 
+# Requirements
+
+* pysaml2 7.1+
+
 # Usage
 
 ```

diff --git a/nagios-check-saml-entity.py b/nagios-check-saml-entity.py
@@ -124,8 +124,8 @@ def nagios_exit(message, code):
 
     if args.saml_cert_days:
         certs = list(set(
-            mds.certs(entity_id=args.entity, descriptor='any', use='encryption') +
-            mds.certs(entity_id=args.entity, descriptor='any', use='signing')))
+            [mds.certs(entity_id=args.entity, descriptor='any', use='encryption')[0][1]] +
+            [mds.certs(entity_id=args.entity, descriptor='any', use='signing')[0][1]]))
         if len(certs) > 0:
             for i in certs:
                 cert = x509.load_der_x509_certificate(base64.b64decode(i), default_backend())