From 78f643f66007d804d071d4868a802bacda85df4c Mon Sep 17 00:00:00 2001 From: Fabien Lamarque Date: Mon, 1 Jul 2024 12:43:25 +0000 Subject: [PATCH] Supprime la variable d'env AVEC_AUTHENTIFICATION_EIDAS Co-authored-by: Emmanuel Gaillot --- .env.site.template | 1 - src/adaptateurs/adaptateurEnvironnement.js | 3 --- src/api/creationSessionFCPlus.js | 2 +- test/api/creationSessionFCPlus.spec.js | 2 -- test/routes/serveurTest.js | 1 - 5 files changed, 1 insertion(+), 8 deletions(-) diff --git a/.env.site.template b/.env.site.template index f03bcd9..7e972dd 100644 --- a/.env.site.template +++ b/.env.site.template @@ -3,7 +3,6 @@ AVEC_ENVOI_COOKIE_SUR_HTTP= # autorise envoi du cookie de session par HTTP avec 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 -AVEC_AUTHENTIFICATION_EIDAS= # si renseigné à `true`, passe par le « bridge eIDAS » pour l'authentification CLE_PRIVEE_JWK_EN_BASE64= # Cle privée utilisée pour déchiffrer les infos utilisateur provenant de eIDAS (données au format JWK, chiffrées en base64) IDENTIFIANT_CLIENT_FCPLUS= # identifiant d'accès au serveur FC+ SECRET_CLIENT_FCPLUS= # secret d'accès au serveur FC+ diff --git a/src/adaptateurs/adaptateurEnvironnement.js b/src/adaptateurs/adaptateurEnvironnement.js index 618cb8f..834ea37 100644 --- a/src/adaptateurs/adaptateurEnvironnement.js +++ b/src/adaptateurs/adaptateurEnvironnement.js @@ -6,8 +6,6 @@ const avecMock = () => process.env.AVEC_MOCK === 'true'; const clePriveeJWK = () => JSON.parse(atob(process.env.CLE_PRIVEE_JWK_EN_BASE64)); -const fournisseurIdentiteSuggere = () => (process.env.AVEC_AUTHENTIFICATION_EIDAS === 'true' ? 'eidas-bridge' : ''); - const identifiantClient = () => process.env.IDENTIFIANT_CLIENT_FCPLUS; const parametresRequeteJeton = () => ({ @@ -29,7 +27,6 @@ module.exports = { avecConnexionFCPlus, avecMock, clePriveeJWK, - fournisseurIdentiteSuggere, identifiantClient, parametresRequeteJeton, secretJetonSession, diff --git a/src/api/creationSessionFCPlus.js b/src/api/creationSessionFCPlus.js index 2bec2bf..be8cb96 100644 --- a/src/api/creationSessionFCPlus.js +++ b/src/api/creationSessionFCPlus.js @@ -13,7 +13,7 @@ const creationSessionFCPlus = (config, requete, reponse) => { : ''; const fournisseurIdentiteSuggere = typeof requete.query.eidas !== 'undefined' - ? adaptateurEnvironnement.fournisseurIdentiteSuggere() + ? 'eidas-bridge' : ''; const construisURL = () => adaptateurFranceConnectPlus.urlCreationSession() diff --git a/test/api/creationSessionFCPlus.spec.js b/test/api/creationSessionFCPlus.spec.js index 5d90f13..c7e9e02 100644 --- a/test/api/creationSessionFCPlus.spec.js +++ b/test/api/creationSessionFCPlus.spec.js @@ -23,7 +23,6 @@ describe('Le requêteur de création de session FC+', () => { adaptateurChiffrement.cleHachage = () => ''; adaptateurChiffrement.genereJeton = () => Promise.resolve(); adaptateurEnvironnement.avecMock = () => false; - adaptateurEnvironnement.fournisseurIdentiteSuggere = () => ''; adaptateurEnvironnement.identifiantClient = () => ''; adaptateurEnvironnement.urlRedirectionConnexion = () => ''; adaptateurFranceConnectPlus.urlCreationSession = () => Promise.resolve(''); @@ -118,7 +117,6 @@ describe('Le requêteur de création de session FC+', () => { it('renseigne le paramètre `idp_hint` avec la valeur `eidas-bridge`', () => { expect.assertions(1); requete.query.eidas = ''; - adaptateurEnvironnement.fournisseurIdentiteSuggere = () => 'eidas-bridge'; prepareVerificationPresenceElement('idp_hint=eidas-bridge', reponse); return creationSessionFCPlus(config, requete, reponse); diff --git a/test/routes/serveurTest.js b/test/routes/serveurTest.js index 1c5964c..61826a8 100644 --- a/test/routes/serveurTest.js +++ b/test/routes/serveurTest.js @@ -27,7 +27,6 @@ const serveurTest = () => { avecConnexionFCPlus: () => true, avecEnvoiCookieSurHTTP: () => true, avecMock: () => true, - fournisseurIdentiteSuggere: () => '', identifiantClient: () => '', secretJetonSession: () => 'secret', urlRedirectionConnexion: () => '',