Skip to content
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] Mise à jour de openid-client en version 6 (PIX-16870) #11586

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Mar 6, 2025

🥞 Problème

Le librarie openid-client est passée en version v6 avec de nombreux breaking changes.

🥓 Proposition

Mettre à jour la librarie openid-client en version 6.3.3

🧃 Remarques

Les tests d'acceptance du fichier oidc-provider.route.test.js utilisaient un mix sur l'utilisation de nock et de mock de la librarie openid-client. Cela a été remplacé par un mock complet de la librarie openid-client.

😋 Pour tester

Pré-requis :

  1. En local, charger le fichier OIDC_PROVIDERS.json
  2. Lancer Caddy pour utiliser les domaines https://app.dev.pix.fr...
  3. Redémarrer l'API

Scénarios pour chaque SSO :

  1. Se connecter au SSO via la page de login
  2. Créer le compte (vérifier la récupération du nom et prénom)
  3. Vérifier l'accès à l'application Pix
  4. Se déconnecter
  5. Se reconnecter au SSO et vérifier qu'on est directement connecté.

Sur app.dev.pix.fr, tester les SSO :

  • France travail
  • CNFPT
  • Caisse Nationale d’Assurance Retraite et CARSAT (CNAV)
  • FER
  • Pro connect
  • Region Pays de la loire

Sur app.dev.pix.org tester le SSO :

  • FWB

Test du SSO Google sur Pix Admin:

  • En préalable, insérer l'utilisateur en base de données avec votre ADRESSE EMAIL:
with rows as (insert into "users" ("firstName", "lastName", "email") 
values ('SuperAdmin', 'SuperAdmin', 'ADRESSE EMAIL') returning "id") 
insert into "pix-admin-roles" ("userId", "role") select id, 'SUPER_ADMIN' from rows;
  • Se connecter sur admin.dev.pix.fr avec Google

@bpetetot bpetetot self-assigned this Mar 6, 2025
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@bpetetot bpetetot requested a review from a team as a code owner March 6, 2025 14:43
@bpetetot bpetetot force-pushed the pix-16870-upgrade-openid-client-to-v6 branch from 75c8aef to 9f39809 Compare March 6, 2025 16:44
@bpetetot bpetetot force-pushed the pix-16870-upgrade-openid-client-to-v6 branch from 2b1eb61 to ce2e594 Compare March 7, 2025 09:14
@bpetetot bpetetot force-pushed the pix-16870-upgrade-openid-client-to-v6 branch from f6c6262 to 238ccde Compare March 7, 2025 14:29
@bpetetot bpetetot added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-acces labels Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants