Bug #22826
global name 'slug' is not defined
Début:
27 mars 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
File "/usr/lib/python2.7/dist-packages/wcs/qommon/saml2.py", line 513, in fill_user_attributes 511 role = Role.resolve(uuid) 512 if not role: > 513 logger.warn('role slug %s is unknown', slug) 514 continue 515 role_ids.append(str(role.id))
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
- Lié à Development #14207: Modifier le provisionning pour retrouver les rôles par uuid, slug ou nom ajouté
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-saml-don-t-crash-logging-unknown-role-uuids-22826.patch 0001-saml-don-t-crash-logging-unknown-role-uuids-22826.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 6 ans
(me demandant quand même si on ne devrait pas considérer ce cas comme une erreur, ie logger.error)
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
Ok pour passer en error() après #19853.
commit bf1d616c0fa782f4659f8774b025d708d2a885ea Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Mar 27 15:39:37 2018 +0200 saml: don't crash logging unknown role uuids (#22826)
Mis à jour par Frédéric Péters il y a environ 6 ans
Quoique. Dans un SSO dans Publik l'assertion contient un attribut role-slug; rempli avec a2_service_ou_role_uuids il reprend les rôles "techniques" genre _a2-hobo-superuser. Mais ces rôles n'étant par ailleurs pas provisionnés, on se trouve avec le log en question. Côté mellon on ne dit rien en cas de rôle inconnu.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
saml: don't crash logging unknown role uuids (#22826)