-
Notifications
You must be signed in to change notification settings - Fork 56
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
[TECH] Simplifier l'usage de stratégie de connexions au sein de l'API (PIX-16585). #11427
[TECH] Simplifier l'usage de stratégie de connexions au sein de l'API (PIX-16585). #11427
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
api/tests/prescription/campaign-participation/acceptance/application/pole-emploi-route_test.js
Outdated
Show resolved
Hide resolved
api/src/prescription/campaign-participation/application/pole-emploi-route.js
Outdated
Show resolved
Hide resolved
f5b0e1a
to
db96aae
Compare
25bafe4
to
61b5d38
Compare
FUNC Review OK : user + application |
…otect routes with scope
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Jérémie Jadé <[email protected]> Co-authored-by: Vincent Hardouin <[email protected]>
61b5d38
to
3c10f7e
Compare
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.
Nickel 👏 Le code est beaucoup plus clair 👍
J'ai testé la connexion utilisateur et application (parcours sup) ✅
🥞 Problème
Actuellement, nous avons une unique stratégie de connexion pour les utilisateurs et une stratégie pour chaque application, servant en réalité de restriction pour les routes.
🥓 Proposition
Nous proposons d'utiliser le scope à bon escient en protégeant les routes avec grâce à
access: { scope: [] }
proposé par Hapi.js.Puis, nous proposons un petit refacto du fichier
authentication
avec deux bénéfices :_checkIsAuthenticated
renommé au passage enauthenticateJWT
agnostique du token et déléguer la partie spécifique à la méthodevalidate
de la stratégie.Enfin, nous avons ajouté une configuration unique pour toutes les applications
🧃 Remarques
😋 Pour tester
Essayer de se connecter en tant qu'utilisateur et en tant qu'application