-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ajout d'un bouton connexion eIDAS #17
Conversation
Co-authored-by: Fabien Lamarque <[email protected]>
Co-authored-by: Fabien Lamarque <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Codé en binôme
@@ -12,8 +12,12 @@ const creationSessionFCPlus = (config, requete, reponse) => { | |||
? `&contexte_mock=${contexteMock}` | |||
: ''; | |||
|
|||
const fournisseurIdentiteSuggere = typeof requete.query.eidas !== 'undefined' | |||
? adaptateurEnvironnement.fournisseurIdentiteSuggere() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ce code doit disparaître… bientôt 🤞
@@ -12,11 +12,13 @@ const routesBase = (config) => { | |||
'/', | |||
(...args) => middleware.renseigneUtilisateurCourant(...args), | |||
(requete, reponse) => { | |||
const avecConnexionEIDAS = process.env.AVEC_AUTHENTIFICATION_EIDAS === 'true'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On espère que cette ligne de code va également disparaître bientôt 🤞
Ce bouton ne s'affiche que si la variable d'environnement
AVEC_AUTHENTIFICATION_EIDAS
est renseignée àtrue
.