Project

General

Profile

Tâche #30230

Scénario #29663: Correspondance mot de passe eole-ad-Scribe et Ad Windows 2019

Utiliser le test dans l’EAD2

Added by Benjamin Bohard over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
04/28/2020
Due date:
% Done:

100%

Remaining (hours):
0.0

Related issues

Related to Distribution EOLE - Tâche #30594: Modification d'API non répercutée dans le script d'ajout d'utilisateur Horus Fermé 09/09/2020

Associated revisions

Revision bf5c06af (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision 45ccaf68 (diff)
Added by Philippe Caseiro over 2 years ago

Using configuration file instead of CreoleGet

ref #30230

Revision 81dd112c (diff)
Added by Philippe Caseiro over 2 years ago

Moving variable to the good section.

Using getVar, this variable does not exists all the time.

ref #30230

Revision ad1c2ccf (diff)
Added by Philippe Caseiro over 2 years ago

Moving variable to a better location.

This variable is better suited for userpassword/dicos/19_password.xml

ref #30230

Revision 43646569 (diff)
Added by Philippe Caseiro over 2 years ago

Moving variable to a better location.

This variable is better suited for userpassword/dicos/19_password.xml

ref #30230

Revision 6c375db7 (diff)
Added by Philippe Caseiro over 2 years ago

Moving variable to a better location.

This variable is better suited for userpassword/dicos/19_password.xml

ref #30230

Revision 8e0de7ba (diff)
Added by Philippe Caseiro over 2 years ago

Moving variable to a better location.

This variable is better suited for userpassword/dicos/19_password.xml

ref #30230

Revision 181bfa70 (diff)
Added by Philippe Caseiro over 2 years ago

Fix test_confirm_passwd usage with new API.

ref #30230

Revision 92951f95 (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision 399cb4f1 (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision 41294c9f (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision d7c0e6e6 (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision 8ded87f3 (diff)
Added by Philippe Caseiro over 2 years ago

Using new passwd.password_is_valid test

If needed check if username is contained in password

ref #30230

Revision 614cca1d (diff)
Added by Philippe Caseiro over 2 years ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision 017bc127 (diff)
Added by Philippe Caseiro over 2 years ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision e15ca1e6 (diff)
Added by Philippe Caseiro over 2 years ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision 7530e4fb (diff)
Added by Philippe Caseiro over 2 years ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision f92b0a82 (diff)
Added by Joël Cuissinat over 2 years ago

Fix regression in teacher creation

Ref: #30230

History

#1 Updated by Philippe Caseiro over 2 years ago

  • Status changed from Nouveau to En cours

#2 Updated by Philippe Caseiro over 2 years ago

  • Status changed from En cours to Nouveau

#3 Updated by Philippe Caseiro over 2 years ago

  • Status changed from Nouveau to En cours

#4 Updated by Philippe Caseiro over 2 years ago

  • Assigned To set to Philippe Caseiro

#5 Updated by Philippe Caseiro over 2 years ago

  • Status changed from En cours to Résolu

#6 Updated by Joël Cuissinat over 2 years ago

  • Status changed from Résolu to En cours

Je viens de tester une modification de mot de passe dans l'EAD en 2.6.2 :

2020-06-11T16:09:05.339537+02:00 scribe.ac-test.fr ead-server[739]: #011Traceback (most recent call last):
2020-06-11T16:09:05.339786+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 317, in addCallback
2020-06-11T16:09:05.339981+02:00 scribe.ac-test.fr ead-server[739]: #011    callbackKeywords=kw)
2020-06-11T16:09:05.340226+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 306, in addCallbacks
2020-06-11T16:09:05.340487+02:00 scribe.ac-test.fr ead-server[739]: #011    self._runCallbacks()
2020-06-11T16:09:05.340759+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 588, in _runCallbacks
2020-06-11T16:09:05.341017+02:00 scribe.ac-test.fr ead-server[739]: #011    current.result = callback(current.result, *args, **kw)
2020-06-11T16:09:05.341256+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/share/ead2/backend/lib/execute_wrapper.py", line 18, in wrapper
2020-06-11T16:09:05.341490+02:00 scribe.ac-test.fr ead-server[739]: #011    return func(self)
2020-06-11T16:09:05.341748+02:00 scribe.ac-test.fr ead-server[739]: #011--- <exception caught here> ---
2020-06-11T16:09:05.341987+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/share/ead2/backend/actions/scribe/userpwd.py", line 230, in execute
2020-06-11T16:09:05.342232+02:00 scribe.ac-test.fr ead-server[739]: #011    result['message'] = self._valid_form()
2020-06-11T16:09:05.342461+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/share/ead2/backend/actions/scribe/userpwd.py", line 177, in _valid_form
2020-06-11T16:09:05.342737+02:00 scribe.ac-test.fr ead-server[739]: #011    resultat['repassword'])
2020-06-11T16:09:05.342978+02:00 scribe.ac-test.fr ead-server[739]: #011  File "/usr/share/ead2/backend/actions/scribe/tool/validate.py", line 34, in test_confirm_passwd
2020-06-11T16:09:05.343227+02:00 scribe.ac-test.fr ead-server[739]: #011    test_password_length(pwd1)
2020-06-11T16:09:05.343449+02:00 scribe.ac-test.fr ead-server[739]: #011exceptions.TypeError: test_password_length() takes exactly 2 arguments (1 given)

#7 Updated by Philippe Caseiro over 2 years ago

  • Status changed from En cours to Résolu

#8 Updated by Joël Cuissinat over 2 years ago

  • Status changed from Résolu to En cours

#9 Updated by Joël Cuissinat over 2 years ago

Je viens de re-tester en 2.6.2.
Il n'y a plus de traceback et si on passe la variable check_for_username à oui, le test du login est bien réalisé.

Mais il y a encore une faille dans l'algo : avec la config par défaut + check_for_username, à partir où le login n'est pas dans le mot de passe, le nombre de classes n'est pas vérifié.

Avec la configuration suivante, le mot de passe aaaaa est accepté (enfin sauf si il y aaaaa dans le login).

smb_min_password_class="2" 
smb_min_password_length="5" 
check_for_username="oui" 

#10 Updated by Joël Cuissinat over 2 years ago

  • Status changed from En cours to Fermé
  • % Done changed from 0 to 100
  • Remaining (hours) set to 0.0
Testé en 2.6.2 et 2.7.1 avec et sans l'option :
  • création prof
  • création administratif
  • création élève
  • création responsable
  • création invité
  • changement de mot de passe d'un utilisateur via la recherche
  • édition groupée de mot de passe
  • modification de son mot de passe (préférences prof)

#11 Updated by Joël Cuissinat about 2 years ago

  • Related to Tâche #30594: Modification d'API non répercutée dans le script d'ajout d'utilisateur Horus added

Also available in: Atom PDF