georchestra / georchestra-gateway

GNU General Public License v3.0
0 stars 5 forks source link

Show error message to OAuth2 user when a matching local account already exists #86

Closed emmdurin closed 4 months ago

emmdurin commented 9 months ago

If a user tries to log with OAuth2 and a local account with the same e-mail address already exists, it should be presented a clear error messages about this conflict, and the user should not considered logged in.

pierrejego commented 8 months ago

Hello, I am just starting testing gateway. Thansk for this work.

In my need, I want the email address to be the user id for connection. And let the user connect from where ever he wanted if he has the same email. For exemple, compagny could have internal specific SSO (when in VPN ) and use the georchestra SSO when only on internet. The idea is to have the same account if he is in or out the compagny. This could be the same for FranceConnect. A already georchestra user could also have a France connect user with the same mail. It could be a good idea to have only one georchestra account.

I know PR is not the place to discuss about this point, but I didn't find issue link to the PR.

Regards

fvanderbiest commented 8 months ago

I know PR is not the place to discuss about this point, but I didn't find issue link to the PR.

Hi Pierre,

Thanks for testing the software and joining the conversation, but it's a bit out of topic here. Suggested places to interact with the community regarding the gateway and its implementation:

Can you relocate your comment to one of the suggested places ? Thanks !

fvanderbiest commented 8 months ago

it's a bit out of topic here

My bad. Not out of topic here.

To my understanding, France Connect does not recommend to use the email as pivot data. See eg point 16 from https://partenaires.franceconnect.gouv.fr/monprojet/recetter/

emmdurin commented 4 months ago

Closed and followed by PR#116 after a change of source branch.