Projet

Général

Profil

Bug #47706

auth saml, attribut absent, IndexError: list index out of range

Ajouté par Sentry Io il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai
Catégorie:
-
Version cible:
-
Début:
15 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/4938/

IndexError: list index out of range
(13 additional frame(s) were not displayed)
...
  File "authentic2_auth_saml/adapters.py", line 78, in provision
    self.provision_a2_attributes(user, idp, saml_attributes)
  File "contextlib.py", line 74, in inner
    return func(*args, **kwds)
  File "authentic2_auth_saml/adapters.py", line 123, in provision_a2_attributes
    if method(user, idp, saml_attributes, mapping):
  File "authentic2_auth_saml/adapters.py", line 157, in action_set_attribute
    return self.set_user_attribute(user, attribute, value)
  File "authentic2_auth_saml/adapters.py", line 163, in set_user_attribute
    value = value[0]

Fichiers

0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch (2,14 ko) 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch Serghei Mihai, 15 octobre 2020 10:46
0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch (2,17 ko) 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch Serghei Mihai, 15 octobre 2020 15:03
0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch (2,21 ko) 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch Serghei Mihai, 15 octobre 2020 15:25
0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch (2,22 ko) 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch Serghei Mihai, 15 octobre 2020 16:31

Révisions associées

Révision 4fed275b (diff)
Ajouté par Serghei Mihai il y a plus de 3 ans

auth_saml: raise error when no saml attribute value received (#47706)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Projet changé de Suivi des traces à Authentic 2
  • Sujet changé de IndexError: list index out of range à auth saml, attribut absent, IndexError: list index out of range
     * /usr/lib/python3/dist-packages/mellon/views.py in authenticate, line 267.
        user = auth.authenticate(saml_attributes=attributes)

Local Vars

attributes=
{'authn_context_class_ref': 'urn:oasis:names:tc:SAML:2.0:ac:classes:PasswordProtectedTransport',
 'authn_instant': datetime.datetime(2020, 10, 15, 6, 15, 50, tzinfo=<isodate.tzinfo.Utc object at 0x7f0085ecedd8>),
 'first_name': [],
 'issuer': 'https://idp.xxx.fr/idp',
 'last_name': [],
 'mail': [],
 'name_id_content': 'xxx',
 'name_id_format': 'urn:oasis:names:tc:SAML:2.0:nameid-format:entity',
 'session_index': '_0ac32682ea81d8bc8502f64548176763b20de5'}
#2

Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

  • Assigné à mis à Serghei Mihai

Le test bien mais manque un pytest.raises() autour je pense. Au passage tu peux remplacer "too much" par "too many".

#3

Mis à jour par Serghei Mihai il y a plus de 3 ans

Done.

Au passage, on continue à recevoir les traces depuis la prod de l'Essonne.
Je vais faire un ticket.

#5

Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée

Thomas qui demande qu'on ait le nom de l'attribut dans 'too many values' je dirai de faire comme ailleurs, ajouter details={'attribute': attribute}.

#6

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch ajouté
  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch ajouté
  • Statut changé de Solution validée à Solution proposée

Ok

#7

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch supprimé
#8

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch supprimé
#10

Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

T'aurais pu faire pareil juste au dessus :)

#12

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch ajouté
#13

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Fichier 0001-auth_saml-raise-error-when-no-saml-attribute-value-r.patch supprimé
#15

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Solution proposée à Solution validée
#16

Mis à jour par Serghei Mihai il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 4fed275ba33b131c2155917a78f721475b32113d (HEAD -> master, origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Oct 15 09:42:34 2020 +0200

    auth_saml: raise error when no saml attribute value received (#47706)
#17

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF