Projet

Général

Profil

Bug #41961

type scopes FC [TypeError: can only concatenate list (not "unicode") to list]

Ajouté par Sentry Io il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
22 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

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

TypeError: can only concatenate list (not "unicode") to list
(2 additional frame(s) were not displayed)
...
  File "django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "django/views/generic/base.py", line 68, in view
    return self.dispatch(request, *args, **kwargs)
  File "authentic2_auth_fc/views.py", line 325, in dispatch
    scopes = self.get_scopes()
  File "authentic2_auth_fc/views.py", line 212, in get_scopes
    return list(set(['openid'] + app_settings.scopes))

Internal Server Error: /fc/callback/

Fichiers

0001-misc-don-t-create-variables-on-get-41961.patch (996 octets) 0001-misc-don-t-create-variables-on-get-41961.patch Frédéric Péters, 22 avril 2020 10:49

Révisions associées

Révision 66094c66 (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

misc: don't create variables on get() (#41961)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Projet changé de Suivi des traces à Plugin FS FranceConnect
  • Hors marché mis à Non
#2

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Projet changé de Plugin FS FranceConnect à Hobo

Le cas précis semble corrigé, app_settings.scopes m'apparait bien comme étant une liste, je vois,

81.249.250.223 - - [22/Apr/2020:10:13:52 +0200] "POST /franceconnect/ HTTP/1.1" 302 0 "https://hobo-dordogne.test.entrouvert.org/franceconnect/" "Mozilla/5.0 (X11; Linux x86_64; rv:75.0) Gecko/20100101 Firefox/75.0" "hobo-dordogne.test.entrouvert.org" [0.086 s]

il y aurait un moment où hobo taperait un scopes sous forme de texte ?

#3

Mis à jour par Frédéric Péters il y a environ 4 ans

Quand je mets juste ["profile"] dans hobo,

hobo=# select * from environment_variable where name = 'SETTING_A2_FC_SCOPES';
 id  |         name         |    value    | service_pk |     last_update_timestamp     | service_type_id | label | auto 
-----+----------------------+-------------+------------+-------------------------------+-----------------+-------+------
 693 | SETTING_A2_FC_SCOPES | ["profile"] |          1 | 2020-04-22 10:31:15.330343+02 |              11 |       | t

j'ai une chaine vide qui s'inscrit dans le hobo.json d'authentic,

        "SETTING_A2_FC_SCOPES": ""
#4

Mis à jour par Frédéric Péters il y a environ 4 ans

(non, c'est juste qu'il ne se mettait pas à jour, le hobo.json était faux d'avant)

#5

Mis à jour par Frédéric Péters il y a environ 4 ans

Une visite sur la page, sans validation, créait cette variable; parce que get_variable() l'enregistrait, sous forme de chaine.

J'étais d'abord parti pour gérer correctement les valeurs par défaut (et ça a donné #41964 en regardant ça) mais je pense que simplement le get_variable ne devrait rien enregistrer.

#6

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

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

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 66094c66e26c43e8b3ac7ecf188497283d3109cf
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Apr 22 10:48:31 2020 +0200

    misc: don't create variables on get() (#41961)
#8

Mis à jour par Frédéric Péters il y a environ 4 ans

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

Formats disponibles : Atom PDF