Redirige vers la destruction de session quand incident d'authentifica…

…tion

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

src/api/connexionFCPlus.js

@@ -13,9 +13,9 @@ const connexionFCPlus = (config, code, requete, reponse) => {
         return stockeDansCookieSession(infos, adaptateurChiffrement, requete);
       }))
     .then(() => reponse.render('redirectionNavigateur', { destination: '/' }))
-    .catch((e) => {
+    .catch(() => {
       requete.session.jeton = undefined;
-      reponse.render('erreur', { descriptionErreur: `Échec authentification (${e.message})` });
+      reponse.render('redirectionNavigateur', { destination: '/auth/fcplus/destructionSession' });
     });
 };
 

test/api/connexionFCPlus.spec.js

@@ -39,18 +39,19 @@ describe('Le requêteur de connexion FC+', () => {
       .then(() => expect(requete.session.jeton).toBeUndefined());
   });
 
-  it("sert une page d'erreur si le nonce retourné est différent du nonce en session", () => {
-    expect.assertions(1);
+  it('redirige vers la destruction de session FC+ si le nonce retourné est différent du nonce en session', () => {
+    expect.assertions(2);
     adaptateurChiffrement.verifieJeton = () => Promise.resolve({ nonce: 'unNonce' });
 
     requete.session.jeton = { nonce: 'abcde' };
     fabriqueSessionFCPlus.nouvelleSession = () => Promise.resolve({
       enJSON: () => Promise.resolve({ nonce: 'oups' }),
     });
 
-    reponse.render = (_nomModelePage, { descriptionErreur }) => {
+    reponse.render = (nomModelePage, { destination }) => {
       try {
-        expect(descriptionErreur).toBe('Échec authentification (nonce invalide)');
+        expect(nomModelePage).toBe('redirectionNavigateur');
+        expect(destination).toBe('/auth/fcplus/destructionSession');
         return Promise.resolve();
       } catch (e) {
         return Promise.reject(e);

test/routes/routesAuth.spec.js

@@ -123,13 +123,14 @@ describe('Le serveur des routes `/auth`', () => {
           .catch(leveErreur);
       });
 
-      it("sert une page d'erreur quand l'authentification échoue", () => {
+      it("redirige vers la destruction de session FranceConnect+ quand l'authentification échoue", () => {
         serveur.fabriqueSessionFCPlus().nouvelleSession = () => Promise.resolve({
           enJSON: () => Promise.reject(new Error('Oups')),
         });
 
         return axios.get(`http://localhost:${port}/auth/fcplus/connexion_apres_redirection?code=unCode&state=unState`)
-          .then((reponse) => expect(reponse.data).toContain('Échec authentification (Oups)'));
+          .then((reponse) => expect(reponse.data).toContain('<meta http-equiv="refresh" content="0; url=\'/auth/fcplus/destructionSession\'">'))
+          .catch(leveErreur);
       });
     });
   });