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 1 year ago. Updated over 1 year 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 1 year 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 1 year ago

Using configuration file instead of CreoleGet

ref #30230

Revision 81dd112c (diff)
Added by Philippe Caseiro over 1 year 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 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago

Fix test_confirm_passwd usage with new API.

ref #30230

Revision 92951f95 (diff)
Added by Philippe Caseiro over 1 year 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 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision 017bc127 (diff)
Added by Philippe Caseiro over 1 year ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision e15ca1e6 (diff)
Added by Philippe Caseiro over 1 year ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision 7530e4fb (diff)
Added by Philippe Caseiro over 1 year ago

Fix note 9 : need to check classes evry time.

ref #30230

Revision f92b0a82 (diff)
Added by Joël Cuissinat over 1 year ago

Fix regression in teacher creation

Ref: #30230

History

#1 Updated by Philippe Caseiro over 1 year ago

  • Status changed from Nouveau to En cours

#2 Updated by Philippe Caseiro over 1 year ago

  • Status changed from En cours to Nouveau

#3 Updated by Philippe Caseiro over 1 year ago

  • Status changed from Nouveau to En cours

#4 Updated by Philippe Caseiro over 1 year ago

  • Assigned To set to Philippe Caseiro

#5 Updated by Philippe Caseiro over 1 year ago

  • Status changed from En cours to Résolu

#6 Updated by Joël Cuissinat over 1 year 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 1 year ago

  • Status changed from En cours to Résolu

#8 Updated by Joël Cuissinat over 1 year ago

  • Status changed from Résolu to En cours

#9 Updated by Joël Cuissinat over 1 year 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 1 year 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 1 year 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