Projet

Général

Profil

Bug #26703

ne pas accepter une chaine puis la traiter comme du json alors qu'en fait, non

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
25 septembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

 id |      name       |             value              | service_pk |    last_update_timestamp     | service_type_id | label | auto 
----+-----------------+--------------------------------+------------+------------------------------+-----------------+-------+------
  7 | email_signature | [fixme] Signature à paramétrer |            | 2018-09-25 11:54:08.02995+00 |                 |       | t

et crash,

  File "/usr/lib/python2.7/dist-packages/hobo/environment/utils.py", line 41, in get_installed_services_dict
    for v in Variable.objects.filter(service_pk__isnull=True))),
  File "/usr/lib/python2.7/dist-packages/hobo/environment/utils.py", line 41, in <genexpr>
    for v in Variable.objects.filter(service_pk__isnull=True))),
  File "/usr/lib/python2.7/dist-packages/hobo/environment/models.py", line 42, in json
    return json.loads(self.value)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

Fichiers

0001-variables-don-t-treat-invalid-json-as-formatted-valu.patch (922 octets) 0001-variables-don-t-treat-invalid-json-as-formatted-valu.patch Frédéric Péters, 25 septembre 2018 14:39

Révisions associées

Révision fab9a4f5 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

variables: don't treat invalid json as formatted values (#26703)

Historique

#1

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

Pas trop sûr, ça marcherait ainsi ici mais peut-être qu'on veut plutôt traiter ce qui commencerait par un " comme du json, et modifier les vues manipulant des variables (comme le paramétrage d'emails) pour taper la chaine sous forme de json.

#2

Mis à jour par Thomas Noël il y a plus de 5 ans

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

Cette modif, ça me va, pas trop envie d'être plus intelligent que cela pour l'instant.

#3

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit fab9a4f56fd249aa916bdf65b945813177db47a1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Sep 25 14:38:43 2018 +0200

    variables: don't treat invalid json as formatted values (#26703)
#4

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

Formats disponibles : Atom PDF