Projet

Général

Profil

Development #25692

cellule json : considérer ok une URL vide

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é à:
-
Version cible:
-
Début:
15 août 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Actuellement on loggue "templated URL ... is empty" en warning mais je me trouve à vouloir charger des données (supplémentaires, "additional-data") d'une URL uniquement si un paramètre a été passé, du coup ma définition de l'URL ressemble à "{% if q_lat }{{passerelle_url}}base-adresse/nominatim/reverse?lat={{ q_lat }}&lon={{ q_lon }}{ endif %}". Je me trouve dès lors avec un tas de logs inutiles.

Ce log particulier a été ajouté dans #19261 quand la prise en charge de templates django a été ajoutée, mais le commentaire dans ce ticket tend plutôt à légitimer l'usage, du coup je serais pour retirer la ligne de log.


Fichiers

Révisions associées

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

json cells: don't log on empty URLs (#25692)

Historique

#1

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

#2

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

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

Allez, zou

#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 c5f7d9ec3aa68b298482f86b9e5b018c71f8a80a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Aug 15 09:55:23 2018 +0200

    json cells: don't log on empty URLs (#25692)
#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