CHANGELOG.md

+## [1.9.2](https://gitlab.ics.muni.cz/perun-proxy-aai/python/perun-proxy-utils/compare/v1.9.1...v1.9.2) (2023-08-23)
+
+
+### Bug Fixes
+
+* check_saml totp form detection, response message with --skip_logout_check argument ([13ba531](https://gitlab.ics.muni.cz/perun-proxy-aai/python/perun-proxy-utils/commit/13ba531fd9b4e462fa37235862337c82eafa9cde))
+
 ## [1.9.1](https://gitlab.ics.muni.cz/perun-proxy-aai/python/perun-proxy-utils/compare/v1.9.0...v1.9.1) (2023-08-23)
 
 

perun/proxy/utils/nagios/check_saml.py

@@ -315,7 +315,7 @@ class SAMLChecker:
             print(response_html)
 
         # MFA
-        if "TOTP" in response_html or "privacyIDEA" in response_html:
+        if "totp" in response_html.lower() or "privacyidea" in response_html.lower():
             if self.args.verbose >= 1:
                 print("MFA is required")
             totp_form_action, totp_form_data = parse_form(response_html)
@@ -521,11 +521,11 @@ class SAMLChecker:
                     "CRITICAL",
                 )
 
-            self.finish(
-                "Authentication took {:.2f} seconds".format(elapsed_seconds),
-                status,
-                auth_time=elapsed_seconds,
-            )
+        self.finish(
+            "Authentication took {:.2f} seconds".format(elapsed_seconds),
+            status,
+            auth_time=elapsed_seconds,
+        )
 
     def __init__(self, args):
         self.args = args

setup.cfg

 [metadata]
-version = 1.9.1
+version = 1.9.2
 license_files = LICENSE
 long_description = file: README.md
 long_description_content_type = text/markdown