Bug #18648
trace connecteur SMS en cas de numéro vide (?)
Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
12 septembre 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Sans doute qu'il est inutile de tracebacker lors de :
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 265, in perform return self.endpoint(request, **self.get_params(request, *args, **kwargs)) File "/usr/lib/python2.7/dist-packages/passerelle/sms/__init__.py", line 55, in send return {'data': self.send_msg(data['message'], data['from'], data['to'], stop=False)} File "/usr/lib/python2.7/dist-packages/passerelle/apps/ovh/models.py", line 92, in send_msg self.default_trunk_prefix) File "/usr/lib/python2.7/dist-packages/passerelle/sms/__init__.py", line 34, in clean_numbers 'trunk prefix)' % number) NotImplementedError: phone number '' is unsupported (no international prefix, no local trunk prefix)
Fichiers
Demandes liées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de Nouveau à Rejeté
Bien d'accord; #18155.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Duplique Bug #18155: sms : ne pas planter sur une chaine vide comme numéro de éléphone ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Fichier 0001-sms-do-not-traceback-on-empty-SMS-number-fixes-18648.patch 0001-sms-do-not-traceback-on-empty-SMS-number-fixes-18648.patch ajouté
- Patch proposed changé de Non à Oui
J'ai fait le minimum mais faudrait faire mieux (genre ne pas appeler la gateway si la liste est vide, pouvoir renvoyer éventuellement une petite erreur et suivre notre standard
{"err": 0/1, ["data": ...]}
Mis à jour par Frédéric Péters il y a plus de 6 ans
Comme tu dis faudrait faire mieux :) (et ici patch proposé sur le ticket rejeté, ça passera inaperçu).