]> git.openstreetmap.org Git - rails.git/commitdiff
Social sign-in: avoid re-authorization in `users_controller#create`
authorMilan Cvetkovic <mcvetkovic@microsoft.com>
Mon, 27 May 2024 14:40:53 +0000 (14:40 +0000)
committerMilan Cvetkovic <mcvetkovic@microsoft.com>
Thu, 30 May 2024 05:43:45 +0000 (05:43 +0000)
It does not add any additional guards against malicious users:

Malicious user may attempt to invoke `POST /users/new` with bogus
values for `auth_provider` and `auth_uid` resulting
with a new account to which user would have a way to login, other than
sending a password reset request.

In some cases, re-authorization would introduce additional
"Please login to your social account", or "Are you sure you want to be logged in"
popup triggered by identity provider.

This PR removes the re-authorization request from `POST /users/new` in authorization flow.


No differences found