Projet

Général

Profil

Bug #22465

opengis: ne pas crasher quand longitude et latitude ne sont pas des float

Ajouté par Thomas Noël il y a environ 6 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Lauréline Guérin
Version cible:
-
Début:
12 mars 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Actuellement :

Error occurred while processing request
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/passerelle/utils/jsonresponse.py", line 117, in api
    resp = f(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/passerelle/views.py", line 276, in perform
    return self.endpoint(request, **self.get_params(request, *args, **kwargs))
  File "/usr/lib/python2.7/dist-packages/passerelle/apps/opengis/models.py", line 153, in feature_info
    bbox = '%s,%s,%s,%s' % (lat, lon, float(lat) + 0.002, float(lon) + 0.002)
ValueError: could not convert string to float: [form_geoloc_base_lat]

parce que l'appel a été fait par w.c.s. sur une demande qui n'a pas de geoloc, ie 'QUERY_STRING': 'lat=%5Bform_geoloc_base_lat%5D&lon=%5Bform_geoloc_base_lon%5D&...

Il faudrait plutôt renvoyer un err:1, ou peut-être une 400, en tout cas ne pas cracher.

Fichiers

0001-opengis-feature_info-bad-params-22465.patch (2,22 ko) 0001-opengis-feature_info-bad-params-22465.patch Lauréline Guérin, 02 octobre 2019 15:12

Révisions associées

Révision 953a9262 (diff)
Ajouté par Lauréline Guérin il y a plus de 4 ans

opengis: feature_info bad params (#22465)

do not crash if lat and/or lon params are not floats

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Assigné à mis à Lauréline Guérin
#2

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

Oui, il ne faut pas cracher c'est pas bien.

#3

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0001-opengis-feature_info-bad-params-22465.patch ajouté
  • Statut changé de Nouveau à Solution proposée
  • Patch proposed changé de Non à Oui
#4

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0001-opengis-feature_info-bad-params-22465.patch ajouté

(j'ai juste changé mon mail en Author)

#5

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0001-opengis-feature_info-bad-params-22465.patch supprimé
#6

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

Je serais plus explicite sur le message d'erreur. Par exemple "Bad coordinates format".

#7

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Fichier 0001-opengis-feature_info-bad-params-22465.patch supprimé
#9

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

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

Fire.

#10

Mis à jour par Lauréline Guérin il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 953a9262cd2cdc7eef1d3d81c63557ba17063d83
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Wed Oct 2 11:19:30 2019 +0200

    opengis: feature_info bad params (#22465)

    do not crash if lat and/or lon params are not floats
#11

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

Formats disponibles : Atom PDF