diff --git a/server/auth/modules/passport-providers/oidc.js b/server/auth/modules/passport-providers/oidc.js index b03343b..9ccebf1 100644 --- a/server/auth/modules/passport-providers/oidc.js +++ b/server/auth/modules/passport-providers/oidc.js @@ -1,7 +1,9 @@ -var OpenIDConnectStrategy = require('passport-openidconnect') -var authUserAssoc = require('../../../models/authUserAssociation') -var users = require('../../../models/users') -var { hasNestedValue } = require('../../../utils') +var OpenIDConnectStrategy = require('passport-openidconnect'); +var authUserAssoc = require('../../../models/authUserAssociation'); +var users = require('../../../models/users'); +var { hasNestedValue } = require('../../../utils'); +const { MISSING_OIDC_PARAMETER } = require('../../../constants/errorCodes.js'); +const AppError = require('../../../middleware/AppError.js'); class PassportOpenIDConnect { constructor(passportjs, auth_name) { @@ -14,8 +16,8 @@ class PassportOpenIDConnect { const config = await fetch(provider.OIDC_CONFIG_URL) return await config.json() } catch (error) { - console.error(`Les informations de connexions de la connexion OIDC ${name} n'ont pu être chargées.`); console.error(`Error: ${error} `); + throw new AppError(MISSING_OIDC_PARAMETER(name)); } } diff --git a/server/constants/errorCodes.js b/server/constants/errorCodes.js index 41147ae..36ae657 100644 --- a/server/constants/errorCodes.js +++ b/server/constants/errorCodes.js @@ -12,6 +12,13 @@ exports.MISSING_REQUIRED_PARAMETER = { code: 400 } +exports.MISSING_OIDC_PARAMETER = (name) => { + return { + message: `Les informations de connexions de la connexion OIDC ${name} n'ont pu être chargées.`, + code: 400 + } +} + exports.USER_ALREADY_EXISTS = { message: 'L\'utilisateur existe déjà.', code: 400