Assertion failed lors des requêtes au LDAP #3

New issue

Open

opened 2018-10-03 11:23:45 +02:00 by mpepin · 9 comments

mpepin commented

2018-10-03 11:23:45 +02:00

(Migrated from git.eleves.ens.fr)

Au début de la fonction utils.get_ldap_infos :

assert clipper_uid.isalnum()

Parfois ce assert échoue sur WikiENS. Est-ce dû à une mauvaise utilisation d'allauth_ens ou bien devrait-on gérer un peu mieux cette erreur ici ?

À noter : le wiki n'utilise pas la dernière version d'allauth_ens, peut-être que le problème a déjà été réglé ?

Au début de la fonction `utils.get_ldap_infos` : ```python assert clipper_uid.isalnum() ``` Parfois ce `assert` échoue sur WikiENS. Est-ce dû à une mauvaise utilisation d'allauth_ens ou bien devrait-on gérer un peu mieux cette erreur ici ? À noter : le wiki n'utilise pas la dernière version d'allauth_ens, peut-être que le problème a déjà été réglé ?

mpepin commented

2018-10-03 11:24:31 +02:00

(Migrated from git.eleves.ens.fr)

@narmanli

delobell commented

2018-10-03 19:23:33 +02:00

(Migrated from git.eleves.ens.fr)

J'ai pas de raison évidente qui me vienne pour expliquer comment on en arrive là.
On pourrait ajouter des logs pour avoir un peu plus de contexte.

J'ai pas de raison évidente qui me vienne pour expliquer comment on en arrive là. On pourrait ajouter des logs pour avoir un peu plus de contexte.

delobell commented

2018-10-03 19:24:32 +02:00

(Migrated from git.eleves.ens.fr)

Ce bout existe encore dans la version 1.1.0... Mettre à jour n'y changera sûrement rien pour WikiENS...

mpepin commented

2018-10-04 19:23:34 +02:00

(Migrated from git.eleves.ens.fr)

Cause probable : les espaces

assert "   ".isalnum()

-> erreur

Cause probable : les espaces ```python assert " ".isalnum() ``` -> erreur

mpepin commented

2018-10-06 12:22:38 +02:00

(Migrated from git.eleves.ens.fr)

mentioned in merge request !13

delobell commented

2018-10-06 17:08:58 +02:00

(Migrated from git.eleves.ens.fr)

Presque @mpepin !
Les tests foirent (sur master) après merge de !13 : https://git.eleves.ens.fr/cof-geek/django-allauth-ens/-/jobs/938 pour py3, mais pas py2 :
https://git.eleves.ens.fr/cof-geek/django-allauth-ens/pipelines/803

Presque @mpepin ! Les tests foirent (sur master) après merge de !13 : https://git.eleves.ens.fr/cof-geek/django-allauth-ens/-/jobs/938 pour py3, mais pas py2 : https://git.eleves.ens.fr/cof-geek/django-allauth-ens/pipelines/803

mpepin commented

2018-10-06 17:19:35 +02:00

(Migrated from git.eleves.ens.fr)

oups…, Je viens de pousser un truc py3-compatible sur Kerl/fix-test

delobell commented

2018-10-06 17:29:32 +02:00

(Migrated from git.eleves.ens.fr)

🍿 devant la CI ?

Edit: ça a l'air d'aller : https://git.eleves.ens.fr/cof-geek/django-allauth-ens/-/jobs/952

:popcorn: devant la CI ? Edit: ça a l'air d'aller : https://git.eleves.ens.fr/cof-geek/django-allauth-ens/-/jobs/952

mpepin commented

2018-10-06 17:40:02 +02:00