Make NameID emailAddress value assertion optional

Abierto
#408 4 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
55/100
Tipo de issue
Nueva funcionalidad
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
php

Línea de trabajo

Empieza inspeccionando la implementación de NameID mencionada en la issue y localiza la aserción que exige que el Subject NameID sea una dirección de correo electrónico. Revisa las pruebas y los patrones de configuración cercanos; el trabajo estará terminado cuando los llamadores puedan desactivar esta aserción para NameIDs que no sean direcciones de correo electrónico, mientras la validación actual sigue siendo el comportamiento predeterminado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

We just updated from v4 to v6 and some authentication failed. After digging around we found that the value of the Subject NameID isn't a valid email address in some cases. This isn't something we can ask the IdP to change.

From what we can see from other SAML2 libraries, it seems to be a common practice to be able to disable some assertions like this one.

https://github.com/simplesamlphp/saml2/blob/85b0530f5f358d8d4cc2f579e271465502516ccc/src/XML/saml/NameID.php#L55-L58

Lenguaje dominante
PHP
Estrellas
306
Forks
138
Métricas de merge de PR
Sin PR fusionados en 30 d

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de simplesamlphp/saml2

Todos los issues de simplesamlphp/saml2

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.