add try catch if call to grist fails (#286)

numerique-gouv · Oct 11, 2024 · 393b962 · 393b962
2 parents b9eb021 + ed6f266
commit 393b962
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 44 deletions.
diff --git a/lib/format.js b/lib/format.js
@@ -86,7 +86,7 @@ module.exports.extractEmailDomain = function(email) {
 
   // Vérification si l'adresse email est valide
   if (!emailRegex.test(email)) {
-      throw new Error("Adresse email invalide");
+    return undefined
   }
 
   // Extraction du domaine avec un split

diff --git a/lib/format.test.js b/lib/format.test.js
diff --git a/lib/oidcAuth.js b/lib/oidcAuth.js
@@ -125,12 +125,12 @@ module.exports.finishAuth = async (req) => {
   try {
     const domain = format.extractEmailDomain(email)
     const whitelistedDomains = await getWhitelistedDomains()
-    if(!whitelistedDomains.includes(domain)){
-      throw new Error(`The domain ${domain} is not whitelisted.`)
+    if(!domain || !whitelistedDomains.includes(domain)){
+      console.error(`The domain ${domain} is not whitelisted.`)
+      return { error: `L'adresse e-mail ${email} n'est pas autorisée à utiliser ce service. Si vous êtes agent de l'État, contactez-nous à [email protected]` }
     }
   } catch(e){
     console.error(`error when validating email ${email}`,e)
-    return { error: `L'adresse e-mail ${email} n'est pas autorisée à utiliser ce service. Si vous êtes agent de l'État, contactez-nous à [email protected]` }
   }
 
   const user = {id_token: tokenSet.id_token, state: request.state}