[TECH] Simplifier l'usage de stratégie de connections au sein de l'API. #11427
+156
−103
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🥞 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