More resilient to errors.

Author: rohe

example/flask_rp/application.py

@@ -16,7 +16,7 @@ def init_oidc_rp_handler(app):
     if _rp_conf.key_conf:
         _kj = init_key_jar(**_rp_conf.key_conf)
         _path = _rp_conf.key_conf['public_path']
-        # removes ./ and / from the begin of the string
+        # removes ./ and / from the begining of the string
         _path = re.sub('^(.)/', '', _path)
     else:
         _kj = KeyJar()

src/idpyoidc/client/client_auth.py

@@ -675,6 +675,8 @@ def single_authn_setup(name, spec):
     else:
         if spec is None:
             cls = get_client_authn_class(name)
+            if cls is None:
+                cls = importer(name)
         elif isinstance(spec, str):
             cls = importer(spec)
         else:

src/idpyoidc/client/oauth2/utils.py

@@ -76,8 +76,12 @@ def pick_redirect_uri(
         if redirect_uris:
             redirect_uri = redirect_uris[0]
         else:
-            logger.error("No redirect_uri")
-            raise MissingRequiredAttribute("redirect_uri")
+            redirect_uris = context.get_preference("redirect_uris", [])
+            if redirect_uris:
+                redirect_uri = redirect_uris[0]
+            else:
+                logger.error("No redirect_uri")
+                raise MissingRequiredAttribute("redirect_uri")
 
     return redirect_uri
 

src/idpyoidc/server/endpoint.py

@@ -229,14 +229,13 @@ def parse_request(
         # Verify that the client is allowed to do this
         auth_info = self.client_authentication(req, http_info, endpoint=self, **kwargs)
 
-        if "client_id" in auth_info:
-            req["client_id"] = auth_info["client_id"]
+        _client_id = auth_info.get("client_id", "")
+        if _client_id:
+            req["client_id"] = _client_id
 
             _auth_method = auth_info.get("method")
             if _auth_method and _auth_method not in ["public", "none"]:
                 req["authenticated"] = True
-
-            _client_id = auth_info["client_id"]
         else:
             _client_id = req.get("client_id")