zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Teste le format d'IP pour la page qui liste les membres utilisant la même IP #6553

Closed philippemilink closed 1 year ago

philippemilink commented 1 year ago

Problème rapporté par Sentry.

Si on passe comme paramètre une adresse IP invalide, l'appel à get_geo_location_from_ip() va lever une exception ValueError, qui n'est pas rattrapée.

Contrôle qualité

Les tests ajoutés passent la CI.

Tester différentes adresses IP, par exemple :

coveralls commented 1 year ago

Coverage Status

coverage: 88.81% (+0.003%) from 88.807% when pulling 1c9bc5e7950f88f62262fcd0edc71a88c5a496f1 on philippemilink:test-ip-format into 630f5e3dd2bc23c7904994adc9fb590a33525ab9 on zestedesavoir:dev.