Supprime l'identifiant eIDAS de la trame ebms

numerique-gouv · May 27, 2024 · 5bc279f · 5bc279f
1 parent 3a6e2db
commit 5bc279f
Show file tree

Hide file tree

Showing 8 changed files with 0 additions and 47 deletions.
diff --git a/.env.oots.template b/.env.oots.template
@@ -1,5 +1,4 @@
 AVEC_REQUETE_PIECE_JUSTIFICATIVE= # active l'API /requete/pieceJustificative avec valeur true
-IDENTIFIANT_EIDAS= # identifiant eIDAS injecté dans les requêtes (tant qu'on ne sait pas le récupérer)
 URL_OOTS_FRANCE= # URL Serveur OOTS-France, ex. https://oots.gouv.fr
 
 DELAI_MAX_ATTENTE_DOMIBUS= # délai maximum d'attente d'une réponse Domibus à une requête envoyée (en millisecondes)

diff --git a/src/adaptateurs/adaptateurEnvironnement.js b/src/adaptateurs/adaptateurEnvironnement.js
@@ -1,8 +1,5 @@
 const avecRequetePieceJustificative = () => process.env.AVEC_REQUETE_PIECE_JUSTIFICATIVE === 'true';
 
-const identifiantEIDAS = () => process.env.IDENTIFIANT_EIDAS;
-
 module.exports = {
   avecRequetePieceJustificative,
-  identifiantEIDAS,
 };
diff --git a/src/api/pieceJustificative.js b/src/api/pieceJustificative.js
@@ -16,7 +16,6 @@ const estErreurMetier = (e) => estErreurAbsenceReponse(e) || estErreurReponseReq
 const pieceJustificative = (
   {
     adaptateurDomibus,
-    adaptateurEnvironnement,
     adaptateurUUID,
     depotPointsAcces,
   },
@@ -37,7 +36,6 @@ const pieceJustificative = (
       codeDemarche,
       destinataire,
       idConversation,
-      identifiantEIDAS: adaptateurEnvironnement.identifiantEIDAS(),
       typeJustificatif,
       previsualisationRequise: (previsualisationRequise === 'true' || previsualisationRequise === ''),
     }))

diff --git a/src/ebms/requeteJustificatif.js b/src/ebms/requeteJustificatif.js
@@ -11,15 +11,13 @@ class RequeteJustificatif extends Message {
       codeDemarche = 'T1',
       destinataire = {},
       idConversation = config.adaptateurUUID.genereUUID(),
-      identifiantEIDAS = 'DK/DE/123123123',
       typeJustificatif = new TypeJustificatif({}),
       previsualisationRequise = false,
     } = {},
   ) {
     super(config, { destinataire, idConversation });
 
     this.codeDemarche = codeDemarche;
-    this.identifiantEIDAS = identifiantEIDAS;
     this.typeJustificatif = typeJustificatif;
     this.previsualisationRequise = previsualisationRequise;
   }
@@ -113,7 +111,6 @@ class RequeteJustificatif extends Message {
       <rim:SlotValue xsi:type="rim:AnyValueType">
         <sdg:Person>
           <sdg:LevelOfAssurance>High</sdg:LevelOfAssurance>
-          <sdg:Identifier schemeID="eidas">${this.identifiantEIDAS}</sdg:Identifier>
           <sdg:FamilyName>Smith</sdg:FamilyName>
           <sdg:GivenName>Jonas</sdg:GivenName>
           <sdg:DateOfBirth>1999-03-01</sdg:DateOfBirth>

diff --git a/src/routes/routesRequete.js b/src/routes/routesRequete.js
@@ -17,7 +17,6 @@ const routesRequete = (config) => {
       pieceJustificative(
         {
           adaptateurDomibus,
-          adaptateurEnvironnement,
           adaptateurUUID,
           depotPointsAcces,
         },

diff --git a/test/api/pieceJustificative.spec.js b/test/api/pieceJustificative.spec.js
@@ -25,7 +25,6 @@ describe('Le requêteur de pièce justificative', () => {
     adaptateurDomibus.envoieMessageRequete = () => Promise.resolve();
     adaptateurDomibus.pieceJustificativeDepuisReponse = () => Promise.resolve();
     adaptateurDomibus.urlRedirectionDepuisReponse = () => Promise.resolve();
-    adaptateurEnvironnement.identifiantEIDAS = () => 'FR/BE/123456789';
     adaptateurUUID.genereUUID = () => '';
     depotPointsAcces.trouvePointAcces = () => Promise.resolve({});
 
@@ -82,30 +81,6 @@ describe('Le requêteur de pièce justificative', () => {
     return pieceJustificative(config, requete, reponse);
   });
 
-  describe('quand `process.env.IDENTIFIANT_EIDAS` vaut `FR/BE/123456789`', () => {
-    let identifiantTemporaire;
-
-    beforeEach(() => {
-      identifiantTemporaire = process.env.IDENTIFIANT_EIDAS;
-      process.env.IDENTIFIANT_EIDAS = 'FR/BE/123456789';
-    });
-
-    afterEach(() => {
-      process.env.IDENTIFIANT_EIDAS = identifiantTemporaire;
-    });
-
-    it('utilise cet identifiant', () => {
-      adaptateurDomibus.envoieMessageRequete = ({ identifiantEIDAS }) => {
-        try {
-          expect(identifiantEIDAS).toEqual('FR/BE/123456789');
-          return Promise.resolve();
-        } catch (e) { return Promise.reject(e); }
-      };
-
-      return pieceJustificative(config, requete, reponse);
-    });
-  });
-
   it('utilise un identifiant de conversation', () => {
     adaptateurUUID.genereUUID = () => '11111111-1111-1111-1111-111111111111';
 

diff --git a/test/ebms/requeteJustificatif.spec.js b/test/ebms/requeteJustificatif.spec.js
@@ -66,17 +66,6 @@ describe("La vue du message de requête d'un justificatif", () => {
     expect(codeDemarche.LocalizedString['@_value']).toBe('T3');
   });
 
-  it("injecte l'identifiant eIDAS spécifié en variable d'environnement", () => {
-    const requeteJustificatif = new RequeteJustificatif(
-      configurationRequete,
-      { identifiantEIDAS: 'BE/FR/123456789' },
-    );
-    const xml = parseXML(requeteJustificatif.corpsMessageEnXML());
-
-    const personne = valeurSlot('NaturalPerson', xml.QueryRequest.Query);
-    expect(personne.Person.Identifier['#text']).toBe('BE/FR/123456789');
-  });
-
   it("injecte l'identifiant de type de justificatif demandé", () => {
     const requeteJustificatif = new RequeteJustificatif(
       configurationRequete,

diff --git a/test/routes/serveurTest.js b/test/routes/serveurTest.js
@@ -24,7 +24,6 @@ const serveurTest = () => {
 
     adaptateurEnvironnement = {
       avecRequetePieceJustificative: () => true,
-      identifiantEIDAS: () => 'FR/BE/123456789',
     };
 
     adaptateurUUID = {