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

Add new URL for France Connect Intégration #111

Closed
wants to merge 1 commit into from

Conversation

lme-atolcd
Copy link
Contributor

fix #109

fix InseeFr#109

Signed-off-by: Laurent Meunier <lme@atolcd.com>
@clement-dufaure
Copy link
Collaborator

To avoid confusion, we should not use "v3" and introduce new environment name instead, and directly add integ and prod of this new environment

So, 3 (x2) environments : V1 (should stop in 2025), FranceConnect V2 and FranceConnect+ V2, all in integ and prod

Current configuration isn't clear with franceConnectV2 which is really urls of franceconnect+
We shoud rename existing V2 environment in FranceConnect+ (probably in other PR)
(and keep old name with Legacy mention in UI ?)

@clement-dufaure
Copy link
Collaborator

I rewrote environments in #114 ,
Someone could dubble check ?

@lme-atolcd
Copy link
Contributor Author

Not tested, but #114 looks good to me.
I'm ok to merge your PR and close mine.

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.

[BUG] new endpoints for integration environment
2 participants