-
Notifications
You must be signed in to change notification settings - Fork 55
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
Conversation
6642a4a
to
a30e51e
Compare
0c61d67
to
dedfad0
Compare
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
6c2ef9c
to
5292fa9
Compare
5292fa9
to
6f381cd
Compare
dedfad0
to
f0abd35
Compare
476b652
to
ce1843f
Compare
.../identity-access-management/infrastructure/repositories/jobs/event-logging-job.repository.js
Show resolved
Hide resolved
api/src/identity-access-management/domain/usecases/update-user-email-with-validation.usecase.js
Show resolved
Hide resolved
api/tests/identity-access-management/unit/application/jobs/event-logging.job-controller.test.js
Outdated
Show resolved
Hide resolved
...access-management/unit/infrastructure/repositories/jobs/event-logging-job.repository.test.js
Outdated
Show resolved
Hide resolved
api/tests/identity-access-management/unit/application/jobs/event-logging.job-controller.test.js
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.
✅ Lu et testé fonctionnellement avec succès avec Firefox
ce1843f
to
334e2be
Compare
api/src/shared/infrastructure/repositories/jobs/job-repository.js
Outdated
Show resolved
Hide resolved
api/tests/identity-access-management/unit/application/jobs/event-logging.job-controller.test.js
Show resolved
Hide resolved
334e2be
to
98c0c8c
Compare
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
api/src/identity-access-management/application/jobs/event-logging.job-controller.js
Show resolved
Hide resolved
98c0c8c
to
31f825f
Compare
🦄 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 ?
api/src/identity-access-management/domain/usecases/update-user-email-with-validation.usecase.js
🌈 Remarques
💯 Pour tester
Préquis: En local, il est nécessaire de démarrer
npm run dev:api
npm run dev:mon-pix
cd api && npm run start:job
cd audit-logger && npm run dev