DGNum/authens
13
0
Fork 0
Code Issues 2 Pull requests Projects Releases 2 Packages Wiki Activity

More idiomatic default settings

Browse source
This commit is contained in:
Martin Pépin 2020-06-20 17:19:42 +02:00
parent 1cdc4a12d6
commit 744f12dccf
No known key found for this signature in database
GPG key ID: E7520278B1774448
2 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
authens/conf.py
View file

@ -1,4 +1,2 @@
default_settings = {
"LDAP_SERVER_URL": "ldaps://ldap.spi.ens.fr:636",
# TODO: CAS_SERVER_URL
}
LDAP_SERVER_URL = "ldaps://ldap.spi.ens.fr:636"
# TODO: CAS_SERVER_URL

4
authens/shortcuts.py
View file

@ -6,7 +6,7 @@ import ldap
from django.conf import settings
from django.contrib.auth import get_user_model
from authens.conf import default_settings
from authens import conf as default_conf
from authens.models import CASAccount, OldCASAccount
from authens.utils import parse_entrance_year
@ -25,7 +25,7 @@ def fetch_cas_account(cas_login):
if not cas_login.isalnum():
raise ValueError("Illegal CAS login: {}".format(cas_login))
ldap_url = getattr(settings, "LDAP_SERVER_URL", default_settings["LDAP_SERVER_URL"])
ldap_url = getattr(settings, "LDAP_SERVER_URL", default_conf.LDAP_SERVER_URL)
ldap_obj = ldap.initialize(ldap_url)
res = ldap_obj.search_s(
"dc=spi,dc=ens,dc=fr",