Authentification JWT sur le backend super-graph #8

wpetit · 2020-06-21T11:31:44+02:00

wpetit commented

2020-06-21 11:31:44 +02:00

Cette PR apporte les modifications suivantes:

Ajout d'une colonne role à la table users pouvant pour l'instant prendre pour valeur user ou admin.
Création de comptes de tests (voir backend/config/seed.js):
- admin@cadoles.com
- user1@cadoles.com
- user2@cadoles.com
- user3@cadoles.com

Attention Penser à faire un make build avant de tester la PR

N'importe quel utilisateur avec une adresse courriel peut se connecter. Cependant, en fonction de l'existence d'un compte utilisateur dans la table users et de la valeur de l'attribut role de celui ci, l'utilisateur connecté aura un rôle différent. Par défaut, une personne se connectant mais n'ayant pas de compte utilisateur aura le rôle anon (voir la documentation de super-graph).

J'ai été obligé de modifier super-graph afin de le rendre compatible avec les jetons générés par hydra. Les modifications en question sont présentes dans la branche hydra-compat du fork. Je ferai des PRs vers le projet upstream pour demander à ce que ces évolutions soient réintégrées. L'image du conteneur de l'environnement de dev est désormais basée sur mon fork.

Cette PR apporte les modifications suivantes: - Ajout d'une colonne `role` à la table `users` pouvant pour l'instant prendre pour valeur `user` ou `admin`. - Création de comptes de tests (voir `backend/config/seed.js`): - `admin@cadoles.com` - `user1@cadoles.com` - `user2@cadoles.com` - `user3@cadoles.com` > **Attention** Penser à faire un `make build` avant de tester la PR N'importe quel utilisateur avec une adresse courriel peut se connecter. Cependant, en fonction de l'existence d'un compte utilisateur dans la table `users` et de la valeur de l'attribut `role` de celui ci, l'utilisateur connecté aura un rôle différent. Par défaut, une personne se connectant mais n'ayant pas de compte utilisateur aura le rôle `anon` ([voir la documentation de super-graph](https://supergraph.dev/docs/security#access-control)). J'ai été obligé de modifier `super-graph` afin de le rendre compatible avec les jetons générés par `hydra`. Les modifications en question sont présentes dans la branche [`hydra-compat`](https://forge.cadoles.com/wpetit/super-graph/src/branch/hydra-compat) du fork. Je ferai des PRs vers le projet upstream pour demander à ce que ces évolutions soient réintégrées. L'image du conteneur de l'environnement de dev est désormais basée sur mon fork.

lseys was assigned by wpetit

2020-06-21 11:31:53 +02:00

lseys was unassigned by wpetit

2020-06-21 11:31:53 +02:00

cmsassot was assigned by wpetit

2020-06-21 11:31:54 +02:00

bgaude was assigned by wpetit

2020-06-21 11:31:56 +02:00

tcornaut was assigned by wpetit

2020-06-21 11:31:57 +02:00

wpetit added a new dependency 2020-06-21 11:32:34 +02:00

#7 Configurer l'authentification JWT sur le backend super-grah