Bug #36819
build passerelle cassé
0%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Fichier 0001-prevent-unassigned-params-variable-reference-36819.patch 0001-prevent-unassigned-params-variable-reference-36819.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 4 ans
Si params est vide là c'est qu'il y a un TypeError levé par self.get_params, ce qui ne devrait pas arriver et cache autre chose.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Et comme sans changements les builds précédents étaient ok, le jeu est de regarder aux dépendances.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Et donc, jsonschema,
File "/home/fred/src/eo/venv1.11/local/lib/python2.7/site-packages/jsonschema/_validators.py", line 200, in pattern not js_regex.compile(patrn).search(instance) File "/home/fred/src/eo/venv1.11/local/lib/python2.7/site-packages/js_regex/_impl.py", line 39, in compile raise TypeError("pattern={!r} must be a unicode string".format(pattern)) TypeError: pattern='^[0-9]{4}-[0-9]{2}-[0-9]{2}T[0-9]{2}:[0-9]{2}$' must be a unicode string
Mis à jour par Frédéric Péters il y a plus de 4 ans
Et donc,
--- a/setup.py +++ b/setup.py @@ -103,7 +103,7 @@ setup(name='passerelle', 'lxml', 'python-dateutil', 'Pillow', - 'jsonschema', + 'jsonschema<3.1', 'zeep < 3.0', 'pycrypto', 'unidecode',
+ un ticket pour qu'il faudrait fonctionner avec la dernière version de jsonschema aussi.
+ peut-être un ticket sur jsonschema pour qu'il lève aussi ValidationError sur cette erreur ? ou l'attraper indépendamment, genre ?
--- a/passerelle/views.py +++ b/passerelle/views.py @@ -340,6 +340,8 @@ class GenericEndpointView(GenericConnectorMixin, SingleObjectMixin, View): validate(data, json_schema) except ValidationError as e: raise APIError(e.message, http_status=400) + except TypeError as e: + raise APIError(e.message, http_status=500) d['post_data'] = data return d
Mis à jour par Frédéric Péters il y a plus de 4 ans
J'ai poussé un < 3.1 dans le dépôt histoire de rétablir les builds.
Mis à jour par Frédéric Péters il y a plus de 4 ans
L'erreur a été notée upstream, au moins https://github.com/Julian/jsonschema/issues/611 et https://github.com/Julian/jsonschema/issues/612 et le commit en question a été reverté, https://github.com/Julian/jsonschema/commit/7ee391c6f94cd43e57806c41340515b5e866727f .
Mis à jour par Frédéric Péters il y a plus de 4 ans
Et il y a depuis eu jsonschema 3.1.1 pour inclure ce revert.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
Ok, sans doute pas de changement nécessaire de notre côté dans ce cas.
Mis à jour par Paul Marillonnet il y a plus de 4 ans
- Statut changé de Solution proposée à Rejeté
Mis à jour par Thomas Noël il y a plus de 4 ans
- Statut changé de Rejeté à Résolu (à déployer)
commit 21e455341dfc81224999b65ca311d550f09468c8 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Oct 10 09:08:24 2019 +0200 misc: limit to jsonschema < 3.1 (#36819)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Lié à Development #37241: Laisser un éventuel TypeError dans self.get_params() percer ajouté
misc: limit to jsonschema < 3.1 (#36819)