Supprime le feature-flip de connexion FranceConnect+

Co-authored-by: Emmanuel Gaillot <[email protected]>

.env.site.template

@@ -1,4 +1,3 @@
-AVEC_CONNEXION_FC_PLUS= # autorise connexion à FC Plus avec valeur true
 AVEC_ENVOI_COOKIE_SUR_HTTP= # autorise envoi du cookie de session par HTTP avec valeur true
 AVEC_MOCK= # autorise l'ajout de contextes de serveur mock FC Plus avec valeur true
 SECRET_JETON_SESSION= # secret utilisé pour chiffrer et déchiffrer le jeton stocké dans le cookie de session

src/adaptateurs/adaptateurEnvironnement.js

@@ -1,5 +1,3 @@
-const avecConnexionFCPlus = () => process.env.AVEC_CONNEXION_FC_PLUS === 'true';
-
 const avecEnvoiCookieSurHTTP = () => process.env.AVEC_ENVOI_COOKIE_SUR_HTTP === 'true';
 
 const avecMock = () => process.env.AVEC_MOCK === 'true';
@@ -24,7 +22,6 @@ const urlRedirectionDeconnexion = () => process.env.URL_REDIRECTION_DECONNEXION;
 
 module.exports = {
   avecEnvoiCookieSurHTTP,
-  avecConnexionFCPlus,
   avecMock,
   clePriveeJWK,
   identifiantClient,

src/routes/routesAuth.js

@@ -70,35 +70,27 @@ const routesAuth = (config) => {
   ));
 
   routes.get('/fcplus/destructionSession', (...args) => middleware.renseigneUtilisateurCourant(...args), (requete, reponse) => {
-    if (adaptateurEnvironnement.avecConnexionFCPlus()) {
-      destructionSessionFCPlus(
-        {
-          adaptateurChiffrement,
-          adaptateurEnvironnement,
-          adaptateurFranceConnectPlus,
-        },
-        requete,
-        reponse,
-      );
-    } else {
-      reponse.status(501).send('Not Implemented Yet!');
-    }
+    destructionSessionFCPlus(
+      {
+        adaptateurChiffrement,
+        adaptateurEnvironnement,
+        adaptateurFranceConnectPlus,
+      },
+      requete,
+      reponse,
+    );
   });
 
   routes.get('/fcplus/creationSession', (requete, reponse) => {
-    if (adaptateurEnvironnement.avecConnexionFCPlus()) {
-      creationSessionFCPlus(
-        {
-          adaptateurChiffrement,
-          adaptateurEnvironnement,
-          adaptateurFranceConnectPlus,
-        },
-        requete,
-        reponse,
-      );
-    } else {
-      reponse.status(501).send('Not Implemented Yet!');
-    }
+    creationSessionFCPlus(
+      {
+        adaptateurChiffrement,
+        adaptateurEnvironnement,
+        adaptateurFranceConnectPlus,
+      },
+      requete,
+      reponse,
+    );
   });
 
   return routes;

src/routes/routesBase.js

@@ -2,7 +2,6 @@ const express = require('express');
 
 const routesBase = (config) => {
   const {
-    adaptateurEnvironnement,
     middleware,
   } = config;
 
@@ -13,11 +12,7 @@ const routesBase = (config) => {
     (...args) => middleware.renseigneUtilisateurCourant(...args),
     (requete, reponse) => {
       const infosUtilisateur = requete.utilisateurCourant;
-      const avecConnexionFCPlus = adaptateurEnvironnement.avecConnexionFCPlus();
-      reponse.render('accueil', {
-        infosUtilisateur,
-        avecConnexionFCPlus,
-      });
+      reponse.render('accueil', { infosUtilisateur });
     },
   );
 

src/siteVitrine.js

@@ -38,10 +38,7 @@ const creeServeur = (config) => {
     middleware,
   }));
 
-  app.use('/', routesBase({
-    adaptateurEnvironnement,
-    middleware,
-  }));
+  app.use('/', routesBase({ middleware }));
 
   const arreteEcoute = (suite) => serveur.close(suite);
 

src/vues/accueil.pug

@@ -12,13 +12,10 @@ block bandeau
     .image-site-vitrine
 
 block page
-  if avecConnexionFCPlus
-    if infosUtilisateur
-      p Félicitations #{infosUtilisateur.afficheToi()}, vous avez réussi à vous authentifier avec eIDAS !
-      a(href = '/auth/fcplus/destructionSession') Déconnexion
-    else
-      div
-        h1 Je me connecte avec le bouton eIDAS
-        a.bouton.eidas(href = '/auth/fcplus/creationSession?eidas')
+  if infosUtilisateur
+    p Félicitations #{infosUtilisateur.afficheToi()}, vous avez réussi à vous authentifier avec eIDAS !
+    a(href = '/auth/fcplus/destructionSession') Déconnexion
   else
-    p Pas d'utilisateur courant
+    div
+      h1 Je me connecte avec le bouton eIDAS
+      a.bouton.eidas(href = '/auth/fcplus/creationSession?eidas')

test/routes/routesAuth.spec.js

@@ -154,17 +154,6 @@ describe('Le serveur des routes `/auth`', () => {
         .then((reponse) => expect(reponse.request.path).toContain('id_token_hint=abcdef'))
         .catch(leveErreur);
     });
-
-    it('retourne une erreur 501 si le feature-flipping est désactivé', () => {
-      expect.assertions(1);
-
-      serveur.adaptateurEnvironnement().avecConnexionFCPlus = () => false;
-
-      return axios.get(`http://localhost:${port}/auth/fcplus/creationSession`)
-        .catch(({ response }) => {
-          expect(response.status).toEqual(501);
-        });
-    });
   });
 
   describe('sur GET /auth/fcplus/creationSession', () => {
@@ -173,16 +162,5 @@ describe('Le serveur des routes `/auth`', () => {
         .then((reponse) => expect(reponse.data).toMatch(/<meta http-equiv="refresh" content="0; url='.*'">/))
         .catch(leveErreur)
     ));
-
-    it('retourne une erreur 501 si le feature-flipping est désactivé', () => {
-      expect.assertions(1);
-
-      serveur.adaptateurEnvironnement().avecConnexionFCPlus = () => false;
-
-      return axios.get(`http://localhost:${port}/auth/fcplus/creationSession`)
-        .catch(({ response }) => {
-          expect(response.status).toEqual(501);
-        });
-    });
   });
 });

test/routes/routesBase.spec.js

@@ -40,17 +40,5 @@ describe('Le serveur des routes `/`', () => {
         })
         .catch(leveErreur);
     });
-
-    it("n'affiche pas le bouton quand le feature flip est désactivé", () => {
-      expect.assertions(2);
-
-      serveur.adaptateurEnvironnement().avecConnexionFCPlus = () => false;
-
-      return axios.get(`http://localhost:${port}/`)
-        .then((reponse) => {
-          expect(reponse.status).toEqual(200);
-          expect(reponse.data).not.toContain('Connexion');
-        });
-    });
   });
 });

test/routes/serveurTest.js

@@ -24,7 +24,6 @@ const serveurTest = () => {
     };
 
     adaptateurEnvironnement = {
-      avecConnexionFCPlus: () => true,
       avecEnvoiCookieSurHTTP: () => true,
       avecMock: () => true,
       identifiantClient: () => '',