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

[BUGFIX] Perte des infos GAR quand la page est rafraichie (PIX-12534) #10774

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Dec 10, 2024

🎄 Problème

Lors du parcours utilisateur GAR, 2 bugs ont été identifiés:

  • BUG 1: Quand l'utilisateur rafraîchit sa page avant la réconcialisation (sur l'écran de landing page et de réconciliation), les informations du GAR sont perdues et on le redirige directement vers la connexion.
  • BUG 2: Quand un utilisateur est déjà réconcilié et qu'il se connecte directement à Pix via le GAR, quand il rafraîchit la page, il est déconnecté

🎁 Proposition

  • Pour le BUG 1, les informations de contexte du GAR vont être stockées en session storage.
  • Pour le BUG 2, il faut restaurer l'authentification GAR quand l'utilisateur recharge la page (comme c'est fait actuellement pour les connexions OIDC).

🧦 Remarques

Un utilitaire SessionStorageKey a été ajouté, il permet de stocker des clés dans le session storage en s'assurant que les valeurs soient correctement parsées quand elles sont lues:

const myKeyStorage = new SessionStorageKey('mykey')

myKeyStorage.set('hello')
myKeyStorage.get() // 'hello'

myKeyStorage.set(123)
myKeyStorage.get() // 123

myKeyStorage.set(undefined)
myKeyStorage.get() // undefined

🎅 Pour tester

Se connecter avec le SSO GAR (faux GAR)

  • Indiquer un samlId aléatoire, le prénom Hermione et le nom Granger
  • Se connecter avec le bouton « Sign in »

BUG 1

  1. Sur la page « Saisissez votre code » indiquer le code campagne SCOBADGE1
  2. Cliquer sur le bouton « Accéder au parcours »
  3. Rafraichir la page

on doit pouvoir faire les étapes suivantes sans souci

  1. Cliquer sur le bouton « Je commence »
  2. Indiquer la date de naissance 05-05-2013
  3. Cliquer sur le bouton « C'est parti ! »
  4. Cliquer sur le bouton « Continuer avec mon compte Pix »
  5. Sur le formulaire « J’ai déjà un compte Pix » indiquer l'adresse email [email protected] et son mdp
  6. Faire et envoyer le parcours
  7. Se déconnecter

BUG 2

  1. Se connecter à nouveau avec le SSO GAR (faux GAR), normalement l'utilisateur est déjà réconcilié et se connecte directement.
  2. Rafraîchir la page

L'utilisateur n'est pas déconnecté

@bpetetot bpetetot self-assigned this Dec 10, 2024
@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 marked this pull request as ready for review December 10, 2024 17:18
@bpetetot bpetetot requested a review from a team as a code owner December 10, 2024 17:18
@bpetetot bpetetot force-pushed the pix-12534-bug-on-refresh-after-gar-auth branch from 37e58c3 to 846d1e8 Compare December 10, 2024 17:32
@P-Jeremy
Copy link
Contributor

P-Jeremy commented Dec 11, 2024

Test func Bug 1 OK ✅

Test func Bug 2 OK ✅

  • Pas de log out au refresh et la bonne page est affichée lors du log out manuel

image

@bpetetot bpetetot force-pushed the pix-12534-bug-on-refresh-after-gar-auth branch from 846d1e8 to 52483ac Compare December 11, 2024 13:52
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

À noter, lorsqu'on j'effectue la procédure de tests, sur le formulaire « Rejoignez l'organisation XXX », en actionnant le bouton « C'est parti ! », on obtient l'appel d’API suivant en erreur :

409 Conflict - POST /api/sco-organization-learners/external

Mais on obtient le même appel en erreur sur la branche dev. Ce n'est donc pas une régression 👍

Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Lu et testé fonctionnellement avec succès avec Firefox

J'ai bien pu reproduire les 2 bugs sur dev et l'absence de ces 2 bugs sur la branche de la PR.

@bpetetot bpetetot force-pushed the pix-12534-bug-on-refresh-after-gar-auth branch 2 times, most recently from 238f36c to 9e8d8d5 Compare December 11, 2024 15:50
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'ai refait un tour et tout me semble bon 👍 😄

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12534-bug-on-refresh-after-gar-auth branch from 9e8d8d5 to 67067dc Compare December 12, 2024 08:08
@pix-service-auto-merge pix-service-auto-merge merged commit d0a68eb into dev Dec 12, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12534-bug-on-refresh-after-gar-auth branch December 12, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants