Skip to content

fix: assurer que tous les query params sont mis dans le success_url de Proconnect#880

Merged
dmc1985 merged 2 commits into
mainfrom
fix/insert-all-query-params-in-next-url
Mar 18, 2026
Merged

fix: assurer que tous les query params sont mis dans le success_url de Proconnect#880
dmc1985 merged 2 commits into
mainfrom
fix/insert-all-query-params-in-next-url

Conversation

@dmc1985

@dmc1985 dmc1985 commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Quand l'utilisateur arrive des emplois à Dora par Nexus, l'url pour Dora qui est mis comme next n'a que le premier query param mtm_campaign et le param op n'est plus présent.

Ça arrive parce que les query params de la route oidc/logged sont interprétés comme ça

{"next": "https://staging.dora.inclusion.gouv.fr/services/cibc-formation-conse-mon-conseil-en-evolu?mtm_campaign=lesemplois", "mtm_kwd": "rechservice-prescriber", "op": "ey..."}

Ce PR fait en sorte que next contient tous les params et pas que le premier.

@dmc1985 dmc1985 requested a review from rsebille March 18, 2026 11:26
Comment thread back/dora/oidc/tests/test_oidc.py Outdated
Comment thread back/dora/oidc/tests/test_oidc.py Outdated
Comment thread back/dora/oidc/views.py
@dmc1985 dmc1985 requested a review from rsebille March 18, 2026 14:03
@dmc1985 dmc1985 merged commit d1b4117 into main Mar 18, 2026
7 checks passed
@dmc1985 dmc1985 deleted the fix/insert-all-query-params-in-next-url branch March 18, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants