KeyError: 'features'

[sentry]

Sentry Issue: OPENREPAIRPLATFORM-C

KeyError: 'features'
(1 additional frame(s) were not displayed)
...
  File "openrepairplatform/event/views/event_manager.py", line 129, in get
    res = super().get(request, *args, **kwargs)
  File "openrepairplatform/mixins.py", line 180, in get
    if point.json()['features']:

[Nasjoe]

Update(/home/jonas/Gits/OpenRepairPlatform/openrepairplatform/mixins.py)
⎿  Added 19 lines, removed 7 lines
173 if postcode and settings.LOCATION:
174 request.session["postcode"] = postcode
175 request.session["location"] = None
176 - point = requests.get(
177 - "https://api-adresse.data.gouv.fr/search/"
178 - f"?q={postcode}&type=municipality"
179 - )
180 - if point.json()['features']:
181 - request.session["location"] = (
182 - point.json()['features'][0]['geometry']['coordinates']
176 + try:
177 + # Appel à l'API adresse du gouvernement pour géolocaliser le code postal
178 + # Call the French government address API to geolocate the postcode
179 + point = requests.get(
180 + "https://api-adresse.data.gouv.fr/search/"
181 + f"?q={postcode}&type=municipality",
182 + timeout=5,
183 )
184 + point.raise_for_status()
185 + features = point.json().get("features", [])
186 + if features:
187 + request.session["location"] = (
188 + features[0]["geometry"]["coordinates"]
189 + )
190 + except (requests.RequestException, ValueError, KeyError):
191 + # Si l'API est indisponible ou retourne une réponse invalide,
192 + # on continue sans géolocalisation.
193 + # If the API is unavailable or returns an invalid response,
194 + # we continue without geolocation.
195 + pass
196
197 if not request.session.get("location", False) and settings.LOCATION:
198 if request.session.get("postcode", False):

● Résumé

Problème : L'appel à api-adresse.data.gouv.fr n'avait aucune gestion d'erreur. Quand l'API renvoie une 504 (timeout), point.json() ne contient pas la clé features → KeyError.

Correction (mixins.py:176-192) :

• Ajout d'un timeout=5 sur la requête
• raise_for_status() pour détecter les erreurs HTTP
• .get("features", []) au lieu d'un accès direct par clé
• try/except pour attraper les erreurs réseau, JSON invalide, ou clés manquantes
• En cas d'échec, on continue sans géolocalisation (l'utilisateur sera redirigé vers l'accueil avec un message demandant un code postal valide)

[Nasjoe]

3126c38