simplesamlphp / simplesamlphp/simplesamlphp-module-ldap
ActiveDirectory connector custom error messages
Personne n'a encore pris cette issue.
- Langage dominant
- PHP
- Étoiles
- 5
- Forks
- 14
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
The 'resolveBindException' function in the Active Directory function never gets called when a password is expired. I made a test change by renaming it 'resolveBindError' (taken from standard LDAP connector class) and changing the signature, and now the code gets called but the error messages aren't defined which cause an error when a password is expired:
SimpleSAML\Error\Exception: Key "RESETPASSWORD" for sequence/mapping with keys "ACSPARAMS...." does not exist in core:loginuserpass.twig
I have tried defining custom errors using the getCustomTitles() and getCustomDescriptions() methods but I'm not doing it right, but now I'm wondering if I'm going down the correct path. Any suggestions?
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Comparez la fonction resolveBindException du connecteur Active Directory avec le comportement de resolveBindError du connecteur LDAP standard, y compris les méthodes getCustomTitles() et getCustomDescriptions(). Suivez la façon dont les erreurs de mot de passe expiré parviennent à core:loginuserpass.twig et vérifiez que les clés et les messages d’erreur résultants sont définis afin que la clé RESETPASSWORD manquante ne provoque plus d’exception.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- php
- Domaine
- authentication
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100