Как правильно сделать отвязывание аккаунта?

Question

[kafkiansky]

В общем, имеется следующее: есть таблица networks и таблица users. При первой авторизации (авторизация только через соц сети) пользователь добавляется и в таблицу networks, и в таблицу users. При привязке аккаунта смотрится, нет ли уже такой соц сети в двух таблицах. Если есть, выкидывается exception, что аккаунт привязан за кем-то. Если нет, добавляем запись только в таблицу networks (связь с users many to one). В результате при входе из какой-то соц сети сначала смотрится на запись в networks и через нее достается юзер в таблице users. Вроде бы все просто. Но как быть с отвязкой, если наличие почты обязательно? То есть идем по условиям: если аккаунт остался один, то просто выкидываем exception, что отвязать нельзя. Но если пользователь хочет отвязать соц сеть, которая хранится в users и networks, а не только в networks, но при этом в networks у него есть еще 2 соц сети, то как отвязываем? Можно, например, взять данные любой другой соц сети из списка и заменить ими запись в users, ну и рефрешить сессию, чтобы юзера не заставлять логиниться по новой. Однако мне кажется, это неправильно. Кто что может посоветовать?

Данные:

Табл. users
uuid | client_id | email | auth_at | updated_at

Табл. networks

uuid | user_id | client_id | email

Примерный хэндлер

if ($command->user->getNetworks()->count() === 1) {
        throw new \DomainException('Нельзя отвязать последнюю соц сеть');
  }

if ($command->user->getClientId() ===
    $this->networkRepository->findOneBy(['clientId' => $command->clientId])
) {
     // Надо понять, что делать в этой итерации
}

 $command->user->detachNetwork(
      $command->network,
      $command->clientId
);

 $this->em->flush();

Answer 1

[Decadal]

Есть понятие Primary Email - это контактная информация, куда направляются все запросы (по смене пароля, уведомления и тд). При вашей архитектуре email в users является таким Primary Email, а в networks - список secondary emails. Не самое красивое решение в итоге, но у вас есть следующие варианты:
1) стянуть связанные соц сети и выбрать какой-нибудь email оттуда

$networks = $this->networkRepository->findBy(['clientId' => $command->clientId]);
foreach ($networks as $net) 
{
// здесь логика по которой вы решаете какой именно secondary email делать primary. 
// если решения нет (например нет ни одного акка с email), выбивать ту же ошибку что и при count === 1
}
// решение есть, значит получен некий email

//задаете новый primary, желательно перед этим проверить что нет юзера с таким праймари
$user->setEmail($email);

2) предложить пользователю самостоятельно выбирать Primary Email в случае если он пытается отвязать аккаунт

Comments

[kafkiansky]

Да, я знаком с этим понятием и решение надумал примерно такое же, какое вы написали, но вот думаю, правильно ли это? Так делают все (или многие)? Просто оно мне кажется каким-то топорным. Ну вот пользователь просто захочет заменить какую-то старую соц сеть (которая основная) на новую, а мы не дадим, потому что не осталось почт среди его привязанных аккаунтов. А если и осталось, то давать выбирать почту юзеру как-то глупо: он может указать старую забытую почту, на которую нет смысла делать рассылку. Может, если есть почта среди привязанных аккаунтов, то прикрепим одну из них, а если нет, то просто попросить пользователя вписать, куда мы отправим подтверждение, а уже из письма мы обновим его данные?

[Decadal]

mad_maximus, это решение навязано архитектурой вида "авторизация только через соцсети". В сущности пользователя всегда должен быть закреплен Primary Email, и если происходит первая авторизация, то вы закрепляете тот email, который указан в этой соцсети, но с чего вы взяли что этот email не забыт и не потерян? У вас нет никаких гарантий, эту проблему можно решить лишь отправкой письма для подтверждения почты. А ещё не все соцсети отдадут вам email, некоторые его не требуют или не предоставляют, что делать в таком случае?
Я бы сказал, что нужно переосмыслить архитектуру и добавить в систему понятие "достоверного основного email", который нужно подтверждать через письмо. На данном этапе для вас достаточно при отвязки соц сети попросить пользователя указать его актуальный email и подтвердить его через письмо, после чего свободно отвязывать что нужно

[kafkiansky]

Decadal, да, проблему того, что соц сети не отдают почту, мы решили: если пользователь входит в первый раз через соц сети и эта соц сеть не отдает почту, то мы просим указать вручную, куда шлем письмо с подтверждением. А уже при привязывании наличие почты необязательно, хоть мы ее и сохраняем, если получили.

[Decadal]

mad_maximus, вы можете использовать ровно тот же сценарий, который вы используете для соц-сети, не отдающей почту.
Как вариант, вы можете забрать email любой другой привязанной соц-сети и сказать пользователю - "это актуальный email?" если да то вы знаете что привязывать, если нет, то просите ввести актуальный email. Как бы мы не крутили эти сценарии, итог получается один - вам следует гарантировать наличие проверенного email в профиле юзера. Неудобство в том, что это приходится делать на отвязывании соц сети, а ведь если бы email был подтвержден ранее, вам бы даже не пришлось с этим морочиться.

[kafkiansky]

Decadal, да, вы правы. Пожалуй, буду просить подтверждение почты.