simplesamlphp / simplesamlphp/simplesamlphp-module-ldap
ActiveDirectory connector custom error messages
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- PHP
- Estrellas
- 5
- Forks
- 14
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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?
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Compara la función resolveBindException del conector de Active Directory con el comportamiento de resolveBindError del conector LDAP estándar, incluidos los métodos getCustomTitles() y getCustomDescriptions(). Traza cómo los errores de contraseñas caducadas llegan a core:loginuserpass.twig y verifica que las claves y los mensajes de error resultantes estén definidos, de modo que la clave RESETPASSWORD que falta ya no provoque una excepción.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- php
- Área
- authentication
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100