https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-10-14T15:56:45ZRedmine Entr’ouvertAuthentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3323962021-10-14T15:56:45ZValentin Deniaud
<ul><li><strong>Assigné à</strong> mis à <i>Valentin Deniaud</i></li></ul> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3323982021-10-14T15:57:57ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/58478">0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/58478/0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch">0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch</a> ajouté</li><li><strong>Tracker</strong> changé de <i>Bug</i> à <i>Development</i></li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3324062021-10-14T16:16:06ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>Pour l'exception prendre une classe plus large, <code>ldap.LDAPError</code>, et rapporter l'erreur dans le log, chaque serveur LDAP ayant un peu tendance à sortir l'erreur qui lui plaît.</p>
<p>--</p>
<p>Pour rendre la chose un peu visible à l'utilisateur, lever une exception nouvelle <code>PasswordChangeError(_('LDAP directory refused the password change'))</code> à récupérer dans authentic2.views.PasswordChange.form_valid() pour afficher un message (pas la peine de le faire pour la vue BO équivalente elle n'est pas capable de changer un mot de passe LDAP).</p> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3331092021-10-19T12:23:02ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/58610">0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/58610/0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch">0001-ldap-do-not-crash-if-password-change-is-not-allowed-.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/58611">0002-ldap-inform-user-when-password-change-has-failed-577.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/58611/0002-ldap-inform-user-when-password-change-has-failed-577.patch">0002-ldap-inform-user-when-password-change-has-failed-577.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>Remarques prises en compte.</p> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3344812021-10-26T09:49:43ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3353162021-11-02T09:05:52ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit ae498bb92a923648396a17af965b411a015b6284
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Oct 19 12:37:09 2021 +0200
ldap: inform user when password change has failed (#57733)
commit 20cbe1f6a5de9ed61d90e41c9a491e264e3491f5
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Oct 14 17:56:56 2021 +0200
ldap: do not crash if password change is not allowed (#57733)</pre> Authentic 2 - Development #57733: crash LDAP sur changement de mot de passehttps://dev.entrouvert.org/issues/57733?journal_id=3355362021-11-02T18:16:37ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>