search

vgarvardt / django-loginza

Django application for Loginza service
http://loginza.ru
38 stars 17 forks source link

loginza_data['nickname'] не всегда присутствует #1

Closed andrexus closed 13 years ago

andrexus commented 13 years ago

при авторизации через google Loginza не всегда возвращает nickname, соответственно вылетает KeyError Exception при попытке обратиться к данному ключу, соответственно создать пользователя тоже не получится

vgarvardt commented 13 years ago

Спасибо за баг репорт. Поправил.

andrexus commented 13 years ago

Спасибо за приложение, надеюсь на дальнейшее развитие. Интегрирую у себя на сайте.

andrexus commented 13 years ago

В данный моменты Вы используете первую часть e-mail адреса в качестве username. Однако есть достаточно большая вероятность дублирования этого поля, что может привести к невозможности создания нового пользователя. Возможно есть смысл использовать комбинацию из email и id.

vgarvardt commented 13 years ago

Вобще проверка на дубль имени пользователя есть. Правда сейчас посмотрел - проверяется только один раз и после модификации используется без повторной проверки. Немного изменил - сейчас проверка гарантирует уникальности имени пользователя. Конечно же есть нюансы, типа конкурентной записи, но пока что они видятся мне маловероятными.