search

marazmiki / django-ulogin

Plug a social authentication feature to your Django application easy!
https://django-ulogin.readthedocs.io/ru/latest/
MIT License
26 stars 19 forks source link

Бэкенд авторизации #15

Closed koodjo closed 12 years ago

koodjo commented 12 years ago

Доброго времени суток!

Хорошо бы использовать бэкенд авторизации из настроек, который прописан в AUTHENTICATION_BACKENDS. Пришлось исправлять на свой в этом месте: https://github.com/marazmiki/django-ulogin/blob/master/django_ulogin/views.py#L93

marazmiki commented 12 years ago

Здравствуйте.

Да, действительно, такой хардкод - не очень хорошая штука. Но к сожалению я не знаю, чем его можно заменить. Брать произвольный элемент кортежа AUTHENTICATION_BACKENDS ничуть не лучше, чем забитое значение: мы ведь не знаем, какой из бэкендов сработал.

Сейчас django-ulogin не реализовывает собственный backend (да и я не уверен, понадобится ли он в дальнейшем), поэтому почти наверняка можно сказать, что аутентификация будет идти через дефолтный. Меньшее из зол.

Если у Вас есть какие-то мысли, как это можно обойти, напишите, пожалуйста. Убедительные аргументы в необходимости написать собственный бэкенд для django-ulogin тоже приветствуются ;-)

koodjo commented 12 years ago

Нет, свой бэкенд это лишнее. В большинстве случаев используется стандартный, либо один самопальный. Несколько бэкендов это очень редкий случай.

Как вариант, сделать настройку, вроде такой: ULOGIN_AUTHENTICATION_BACKEND, где можно будет прописать один свой бэкенд, при отсутствии значения, брать стандартный.

marazmiki commented 12 years ago

Окей, тогда договорились :-)

marazmiki commented 12 years ago

Если есть желание, можете обновляться =)

koodjo commented 12 years ago

Спасибо)