Merge pull request #3 from reciprocity/ds/INFRA-1072

ds/INFRA-1072

Author: Nyefan

.github/workflows/python-package.yml

@@ -18,7 +18,6 @@ jobs:
       fail-fast: false
       matrix:
         python-version:
-          - "3.7"
           - "3.8"
           - "3.9"
           - "3.10"

README.md

@@ -95,7 +95,7 @@ Installation
 
 ### Dependencies ###
 
- * python => 3.7
+  * python => 3.8
  * [xmlsec](https://pypi.python.org/pypi/xmlsec) Python bindings for the XML Security Library.
  * [lxml](https://pypi.python.org/pypi/lxml) Python bindings for the libxml2 and libxslt libraries.
  * [isodate](https://pypi.python.org/pypi/isodate) An ISO 8601 date/time/

pyproject.toml

@@ -23,7 +23,6 @@ keywords = [
     "identity",
 ]
 classifiers = [
-    "Programming Language :: Python :: 3.7",
     "Programming Language :: Python :: 3.8",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
@@ -32,11 +31,11 @@ classifiers = [
     "Topic :: Software Development :: Libraries :: Python Modules",
 ]
 dependencies = [
-    "lxml>=4.6.5,!=4.7.0",
-    "xmlsec>=1.3.9",
+    "lxml>=6.0.0",
+    "xmlsec>=1.3.14",
     "isodate>=0.6.1",
 ]
-requires-python = ">=3.7"
+requires-python = ">=3.8"
 
 [project.urls]
 Homepage = "https://saml.info"

src/python3_saml/saml2/idp_metadata_parser.py

@@ -54,10 +54,7 @@ def get_metadata(
         ctx = None
         if not validate_cert:
             if cafile or capath:
-                raise ValueError(
-                    "Specifying 'cafile' or 'capath' while disabling certificate "
-                    "validation is contradictory."
-                )
+                raise ValueError("Specifying 'cafile' or 'capath' while disabling certificate " "validation is contradictory.")
             ctx = ssl.create_default_context()
             ctx.check_hostname = False
             ctx.verify_mode = ssl.CERT_NONE

tests/src/Python3_Saml/saml2_tests/settings_test.py

@@ -97,18 +97,18 @@ def testLoadSettingsFromDict(self):
 
         # test if the cert-file is loaded correct with the default filename
         settings_info = self.loadSettingsJSON()
-        settings_info['security']['nameIdEncrypted'] = True
-        del settings_info['idp']['x509cert']
+        settings_info["security"]["nameIdEncrypted"] = True
+        del settings_info["idp"]["x509cert"]
         settings_7 = OneLogin_Saml2_Settings(settings_info)
         self.assertEqual(len(settings_7.get_errors()), 0)
 
         # test if the cert-file is loaded correct with a custom filename
-        settings_info['idp']['cert_filename'] = "Test_Root_CA.crt"
+        settings_info["idp"]["cert_filename"] = "Test_Root_CA.crt"
         settings_8 = OneLogin_Saml2_Settings(settings_info)
         self.assertEqual(len(settings_8.get_errors()), 0)
 
         # test for the correct error, if there is no cert at all
-        settings_info['idp']['cert_filename'] = "not_existing_file.crt"
+        settings_info["idp"]["cert_filename"] = "not_existing_file.crt"
         try:
             settings_9 = OneLogin_Saml2_Settings(settings_info)
             self.assertNotEqual(len(settings_9.get_errors()), 0)