-
Notifications
You must be signed in to change notification settings - Fork 51
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] Migrer la route POST /api/oidc/token vers src/identity-access-management (PIX-12524) #8926
[TECH] Migrer la route POST /api/oidc/token vers src/identity-access-management (PIX-12524) #8926
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 : |
1621c9f
to
1a87a7e
Compare
api/src/identity-access-management/application/oidc-provider/oidc-provider.controller.js
Outdated
Show resolved
Hide resolved
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.
Test en local sur firefox ✅
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.
Lu et testé avec succès sur Firefox
to identity-access-management bounded context
to identity-access-management bounded context
code from controller to usecase
to identity-access-management bounded context
to remove authentication and user-account directories, and add identity-access-management directory
Co-authored-by: Emmanuelle Bonnemay <emmanuelle.bonnemay@pix.fr>
1a87a7e
to
c3f4709
Compare
🦄 Problème
La route
POST /api/oidc/token
est encore dans le dossierlib
.🤖 Proposition
Faire la migration vers son bounded context
identity-access-management
.🌈 Remarques
Suppression de code mort. Après l'ajout de la librairie
node-openid-client
, nous n'avons plus besoin que les applications fournissent une redirect_uri à l'api.La PR suivante doit être validée avant :
💯 Pour tester