Skip to content

#4851 - Ne plus envoyer les infos liées à l'utilisateur FT connecté dans l'url du formulaire de convention#5067

Open
celineung wants to merge 8 commits into
mainfrom
4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention

Hidden character warning

The head ref may contain hidden characters: "4851-supprimer-les-infos-li\u00e9es-\u00e0-lutilisateur-ft-connect\u00e9-de-lurl-du-formulaire-de-convention"
Open

#4851 - Ne plus envoyer les infos liées à l'utilisateur FT connecté dans l'url du formulaire de convention#5067
celineung wants to merge 8 commits into
mainfrom
4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention

Conversation

@celineung

@celineung celineung commented May 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Checklist avant RfR

État de la PR

  • Le titre respecte la convention : #ID_ISSUE - Message clair et en français, compréhensible par quelqu'un du métier
  • L'issue est en RfR dans le projet GitHub
  • Auteurs assignés sur la PR et l'issue liée
  • Description du travail commentée sur l'issue (cf. /document-issue)
  • Pipeline CI ✅

Revue du code (self-review)

  • Commits propres (pas de WIP, squash si nécessaire)
  • Relecture complète du diff effectuée
  • Pas de console.log, TODO ou code de debug restant

Points d'attention pour le reviewer

Reste à faire :

Suggestion d'autres PR :

  • renommage du provider 'peConnect' -> 'ftConnect'
  • voir si possible d'avoir une seule url de redirection sur les connexions en succès au lieu d'avoir /inclusion-connect-after-login + /pe-connect
  • params d'urls non utilisés (https://immersion-facile.beta.gouv.fr/demande-immersion?fedIdProvider=peConnect&fedId=AuthFailed&fedIdToken=123 --> il ne se passe rien) --> voir avec Guillaume
  • quelques tests manuels de HttpFtConnectGateway.manual.test.ts sont cassés, qui ne sont pas liés à cette PR
  • remonter back/src/domains/core/authentication/ft-connect/port/FtConnectGateway.ts au même niveau que l'auth ProConnect/email --> à voir plus tard, car ProConnectOAuthGateway est dans /connected-user

@celineung celineung linked an issue May 26, 2026 that may be closed by this pull request
Ne plus envoyer les infos liées à l'utilisateur FT connecté dans l'url du formulaire de convention #4851
Open
@github-actions

github-actions Bot commented May 26, 2026

Copy link
Copy Markdown

📌 Deploy a review app for this PR:

➡️ Click here to deploy review app

When clicking the link above:

  1. Click the "Run workflow" button
  2. Enter this PR number: 5067
  3. Click "Run workflow" to start the deployment

@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 3 times, most recently from ce20aad to 2fdb6b5 Compare May 28, 2026 09:03
@tpicaud tpicaud force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch from 2fdb6b5 to 4555c8b Compare May 28, 2026 10:09
@celineung celineung changed the title #4851 - supprimer les infos liées à lutilisateur ft connecté de lurl du formulaire de convention #4851 - Supprimer les infos liées à l'utilisateur FT connecté de l'url du formulaire de convention Jun 2, 2026
@tpicaud tpicaud force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 3 times, most recently from 805e1e7 to e215f4f Compare June 3, 2026 13:21
@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 8 times, most recently from 273a6cc to 064dc6b Compare June 4, 2026 12:04
@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 7 times, most recently from 68a6145 to ef50e5d Compare June 4, 2026 15:17
celineung
celineung commented Jun 4, 2026
View reviewed changes
Comment thread back/src/adapters/primary/routers/auth/auth.e2e.test.ts Outdated
body: {},
status: 302,
headers: {
location: `${appConfig.immersionFacileBaseUrl}/${frontRoutes.error}?kind=peConnectNoAuthorisation`,

@celineung celineung Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

error route ?

@celineung celineung Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

modifié pour : ${appConfig.immersionFacileBaseUrl}/${legacyFrontRoutes.error}
sans le kind car non utilisé

@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch from 159e93b to 74ca401 Compare June 4, 2026 15:47
@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 3 times, most recently from fd077ee to e9b1e70 Compare June 5, 2026 09:27
@celineung celineung changed the title #4851 - Supprimer les infos liées à l'utilisateur FT connecté de l'url du formulaire de convention #4851 - Ne plus envoyer les infos liées à l'utilisateur FT connecté dans l'url du formulaire de convention Jun 5, 2026
@celineung celineung marked this pull request as ready for review June 5, 2026 10:39
bbohec
bbohec requested changes Jun 11, 2026
View reviewed changes

@bbohec bbohec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'ai l'impression que ça mérite un split façon travail préparatoire pour réduire la taille de la PR car même avec une grosse relecture et le manque de recette possible, le risque de casser FtConnect est important.

Comment thread back/src/adapters/primary/routers/auth/createAuthRouter.ts
);

authSharedRouter.afterOAuthLogin(async (req, res) => {
authSharedRouter.afterEmailOrProConnectOAuthLogin(async (req, res) => {

@bbohec bbohec Jun 11, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comme discuté ensemble je préfère qu'on conserve le terme générique oAuth de cette route.
D'autant qu'à terme, une fois confirmé auprès de FT.IO, on pourra utiliser cette route et supprimer authSharedRouter.afterFTConnectOAuthLogin

Mais si tu veux on peut en parler avec le reste de l'équipe pour voir ce qu'ils préfèrent.

Si on conserve ton nommage, on risque de renommer à nouveau.

Comment thread back/src/adapters/primary/routers/auth/createAuthRouter.ts
Comment thread back/src/config/bootstrap/magicLinkUrl.ts Outdated
Comment thread .../domains/core/authentication/connected-user/adapters/oauth-gateway/ProConnectOAuthGateway.ts Outdated
Comment thread .../core/authentication/ft-connect/adapters/InMemoryConventionFranceTravailAdvisorRepository.ts
Comment thread ...tion/ft-connect/use-cases/NotifyFranceTravailUserAdvisorOnConventionFullySigned.unit.test.ts Outdated
Comment thread ...ains/core/authentication/ft-connect/use-cases/BindConventionToFederatedIdentity.unit.test.ts Outdated
Comment thread back/src/domains/core/authentication/connected-user/use-cases/AfterOAuthSuccess.ts Outdated
Comment thread back/src/domains/core/authentication/connected-user/use-cases/AfterOAuthSuccess.ts
Comment thread back/src/domains/core/authentication/connected-user/use-cases/AfterOAuthSuccess.ts Outdated
@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch 9 times, most recently from 0befe3e to 1cf0e1b Compare June 16, 2026 10:29
bbohec
bbohec approved these changes Jun 16, 2026
View reviewed changes
Comment thread back/src/config/helpers/handleRedirectResponseError.ts
@bbohec bbohec force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch from 1cf0e1b to a79d41c Compare June 17, 2026 08:12
@celineung celineung force-pushed the 4851-supprimer-les-infos-liées-à-lutilisateur-ft-connecté-de-lurl-du-formulaire-de-convention branch from a79d41c to f257ed2 Compare June 17, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bbohec bbohec bbohec approved these changes

Assignees

@celineung celineung

@tpicaud tpicaud

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Ne plus envoyer les infos liées à l'utilisateur FT connecté dans l'url du formulaire de convention

3 participants

@celineung @bbohec @tpicaud