Bug #7612
NameIDPolicy is optional
100%
Description
Line 504 of saml2_endpoints.py try to dump the NameIDPolicy node of AuthnRequest uncondtionnaly, as this element is optional, this is a bug.
Fichiers
Demandes liées
Révisions associées
Revert "Do not unconditionnaly dump the NameIDPolicy node as it is optional (#7612)"
This reverts commit fde8ff574fa83268aa79566b1d80944998efb8e9.
NameIDPolicy is optional in AuthnRequest (fixes #7612)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Fichier 0001-Do-not-unconditionnaly-dump-the-NameIDPolicy-node-as.patch 0001-Do-not-unconditionnaly-dump-the-NameIDPolicy-node-as.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Ritesh Patel il y a presque 9 ans
after the above patch, it seems error happens on line 525 in src/authentic2/idp/saml/saml2_endpoints.py.
if you fix line 525, then it complains on line 706.
I think if no NameIDPolicy found in request, then we should assign the default which will fix the issue.
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|fde8ff574fa83268aa79566b1d80944998efb8e9.
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Statut changé de Résolu (à déployer) à Nouveau
- % réalisé changé de 100 à 0
Patch did not fix the problem.
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit authentic2|66e4d0f70b9c9938eb42cc6a79e79b33a9183ab4.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Solution déployée à Fermé
Do not unconditionnaly dump the NameIDPolicy node as it is optional (fixes #7612)