fix: Session params

Session params was introduced here:
454f894

but this commit break it
cf1450a#diff-6c1cca6748acc9f2583d0fca54d4bbbfc92e8836711cf318ca09d3cd4e60f00dR204

in this commit we fixed this feature and also added a unit test to prevent this kind of regressions for the future.

Author: peppelinux

src/oidcop/__init__.py

@@ -1,6 +1,6 @@
 import secrets
 
-__version__ = "2.2.0"
+__version__ = "2.2.1"
 
 DEF_SIGN_ALG = {
     "id_token": "RS256",

src/oidcop/configure.py

@@ -203,7 +203,7 @@ class EntityConfiguration(Base):
         "httpc_params": {},
         "issuer": "",
         "keys": None,
-        "session_key": None,
+        "session_params": None,
         "template_dir": None,
         "token_handler_args": {},
         "userinfo": None,

tests/op_config.json

@@ -269,19 +269,19 @@
     }
   },
   "session_params": {
-      "password": "__password_used_to_encrypt_access_token_sid_value",
-      "salt": "salt involved in session sub hash ",
+      "password": "ses_key",
+      "salt": "ses_salt",
       "sub_func": {
         "public": {
           "class": "oidcop.session.manager.PublicID",
           "kwargs": {
-            "salt": "sdfsdfdsf"
+            "salt": "mysalt"
           }
         },
         "pairwise": {
           "class": "oidcop.session.manager.PairWiseID",
           "kwargs": {
-            "salt": "sdfsdfsdf"
+            "salt": "mysalt"
           }
         }
      }

tests/test_00_configure.py

@@ -37,6 +37,8 @@ def test_op_configure():
     args = dict(configuration.items())
     assert "add_on" in args
 
+    assert "session_params" in configuration
+
 
 def test_op_configure_from_file():
     configuration = create_from_config_file(

tests/test_06_session_manager.py

@@ -73,6 +73,10 @@ def create_session_manager(self):
                 },
                 "token_endpoint": {"path": "{}/token", "class": Token, "kwargs": {}},
             },
+            "session_params": {
+              "password": "ses_key",
+              "salt": "ses_salt"
+            },
             "template_dir": "template",
             "claims_interface": {"class": "oidcop.session.claims.ClaimsInterface", "kwargs": {}},
             "userinfo": {
@@ -104,6 +108,11 @@ def _create_session(self, auth_req, sub_type="public", sector_identifier=""):
             ae, authz_req, USER_ID, client_id=client_id, sub_type=sub_type
         )
 
+    def test_session_manager_salt_key(self):
+        sman = self.session_manager
+        assert sman.key == "ses_key"
+        assert sman.salt == "ses_salt"
+
     @pytest.mark.parametrize(
         "sub_type, sector_identifier",
         [("pairwise", "https://all.example.com"), ("public", ""), ("ephemeral", "")],

tests/test_23_oidc_registration_endpoint.py

@@ -77,7 +77,9 @@ def create_endpoint(self):
         conf = {
             "issuer": "https://example.com/",
             "password": "mycket hemligt",
-            "verify_ssl": False,
+            "httpc_params": {
+                "verify": False, "timeout": 1
+            },
             "capabilities": {
                 "subject_types_supported": ["public", "pairwise", "ephemeral"],
                 "grant_types_supported": [