cleaning code and debug cas inpn

PnX-SI · Jul 16, 2024 · 62f28ba · 62f28ba
1 parent 6e8a31a
commit 62f28ba
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 15 deletions.
diff --git a/src/pypnusershub/auth/providers/cas_inpn_provider.py b/src/pypnusershub/auth/providers/cas_inpn_provider.py
@@ -16,7 +16,6 @@
 
 
 class AuthenficationCASINPN(Authentication):
-    name = "CAS_INPN_PROVIDER"
     label = "INPN"
     is_external = True
     logo = "<i class='fa fa-paw' aria-hidden='true'></i>"
@@ -54,20 +53,15 @@ def authorize(self):
             user = xml_dict["cas:serviceResponse"]["cas:authenticationSuccess"][
                 "cas:user"
             ]
-
         if not user:
             log.info("Erreur d'authentification lié au CAS, voir log du CAS")
             log.error("Erreur d'authentification lié au CAS, voir log du CAS")
-            return render_template(
-                "cas_login_error.html",
-                cas_logout=self.URL_LOGOUT,
-                url_geonature=current_app.config["URL_APPLICATION"],
-            )
+            return redirect(self.logout_url)
 
         ws_user_url = f"{self.URL_INFO}/{user}/?verify=false"
         response = requests.get(
             ws_user_url,
-            (
+            auth=(
                 self.WS_ID,
                 self.WS_PASSWORD,
             ),
@@ -144,9 +138,6 @@ class CASINPNConfiguration(ProviderConfigurationSchema):
             URL_VALIDATION = fields.String(
                 load_default="https://inpn.mnhn.fr/auth/serviceValidate"
             )
-            URL_AUTHORIZE = fields.String(
-                load_default="https://inpn.mnhn.fr/authentication/"
-            )
             URL_INFO = fields.String(
                 load_default="https://inpn.mnhn.fr/authentication/information",
             )

diff --git a/src/pypnusershub/auth/providers/openid_provider.py b/src/pypnusershub/auth/providers/openid_provider.py
@@ -16,7 +16,6 @@ class OpenIDProvider(Authentication):
 
     """
 
-    name = "OPENID_PROVIDER_CONFIG"
     logo = '<i class="fa fa-sign-in"></i>'
     is_external = True
     """
@@ -106,8 +105,6 @@ class OpenIDConnectProvider(OpenIDProvider):
 
     """
 
-    name = "OPENID_CONNECT_PROVIDER_CONFIG"
-
     def revoke(self):
 
         if not "openid_token_resp" in session:

diff --git a/src/pypnusershub/auth/providers/usershub_provider.py b/src/pypnusershub/auth/providers/usershub_provider.py
@@ -13,7 +13,6 @@ class ExternalUsersHubAuthProvider(Authentication):
     Authentication provider for Flask application using UsersHub-authentification-module.
     """
 
-    name = "EXTERNAL_USERSHUB_PROVIDER_CONFIG"
     logo = '<i class="fa fa-users"></i>'
     is_external = False