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

[FEATURE] Journaliser le changement d'adresse email (PIX-14360) #10137

Merged
merged 3 commits into from
Sep 24, 2024

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Sep 18, 2024

🦄 Problème

Actuellement, nous n’avons pas de suivi des opérations effectuées (par qui, quand?) sur les données sensibles des utilisateurs comme le changement d’adresse email.

🤖 Proposition

Lorsque un utilisateur change de lui-même son adresse email, via la gestion de compte (“Mon compte”), on souhaite réaliser une journalisation de l’opération.

Que souhaites-t’on journaliser quand l’utilisateur change l’adresse email ?

client         // PIX_APP
action         // EMAIL_CHANGED
role           // USER
userId         // Id of the current user
targetUserId   // Id of the current user
occurredAt     // date de changement de l'adresse email
data           // { oldEmail, newEmail }
  • Création d’un job de journalisation “générique”.
  • Appeler le job de journalisation dans le use-case le api/src/identity-access-management/domain/usecases/update-user-email-with-validation.usecase.js

🌈 Remarques

  • Les tests unitaires du usecase ont été migrés en test d'intégration.

💯 Pour tester

Préquis: En local, il est nécessaire de démarrer

  • API: npm run dev:api
  • Pix App: npm run dev:mon-pix
  • Workers de Job: cd api && npm run start:job
  • Audit logger: cd audit-logger && npm run dev
  1. Se connecter sur Pix App.
  2. Aller sur "Mon compte"
  3. Changer son adresse email (avec vérification de code) - On peut utiliser MailPit en local pour récupérer le code (http://localhost:8025)
  4. Vérifier que le changement a été journalisé dans l’audit logger avec toutes les informations (directement dans le BDD de l'audit logger):
select * from "audit-log";

@bpetetot bpetetot self-assigned this Sep 18, 2024
@bpetetot bpetetot force-pushed the pix-14360-log-email-changed branch 2 times, most recently from 0c61d67 to dedfad0 Compare September 18, 2024 13:21
@bpetetot bpetetot changed the base branch from dev to pix-14362-add-data-to-log September 18, 2024 13:21
@bpetetot bpetetot marked this pull request as ready for review September 18, 2024 13:25
@bpetetot bpetetot requested a review from a team as a code owner September 18, 2024 13:25
@bpetetot bpetetot changed the base branch from pix-14362-add-data-to-log to dev September 18, 2024 13:26
@bpetetot bpetetot changed the base branch from dev to pix-14362-add-data-to-log September 18, 2024 13:26
@bpetetot bpetetot closed this Sep 19, 2024
@bpetetot bpetetot reopened this Sep 19, 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 :

Base automatically changed from pix-14362-add-data-to-log to dev September 19, 2024 09:41
@bpetetot bpetetot force-pushed the pix-14360-log-email-changed branch 2 times, most recently from 476b652 to ce1843f Compare September 19, 2024 15:10
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

Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay 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é avec succès

@pix-service-auto-merge pix-service-auto-merge merged commit 50e2b57 into dev Sep 24, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14360-log-email-changed branch September 24, 2024 08:03
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