Projet

Général

Profil

Bug #44267

Ne pas lancer d'alerte quand OVH error: {u'status': 202, u'message': u'Following receivers phone number are invalid : "+339..."'}

Ajouté par Thomas Noël il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Rejeté
Priorité:
Bas
Assigné à:
Version cible:
-
Début:
19 juin 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Actuellement on reçoit un mail d'alerte quand il y a tentative d'envoi d'un SMS vers un numéro qui ne peut en recevoir, ce n'est pas pertinent.

https://sentry.entrouvert.org/entrouvert/publik/issues/2608/

Historique

#1

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

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

Mis à jour par Thomas Noël il y a presque 4 ans

Pour info, la doc https://docs.ovh.com/fr/sms/envoyer_des_sms_depuis_une_url_-_http2sms/ indique que les 2xx sont des erreurs (ne pas rire).

    100 ou 101 : requête traitée.
    201 : un paramètre est manquant (exemples : Missing login, Missing password).
    202 : un paramètre est incorrect (exemples : Invalid tag: is too long, Invalid deferred time).
    401 : pas d’IP autorisée. Pour gérer les IP autorisées, vous pouvez appliquer des restrictions depuis votre espace-client.

Et donc il faudrait peut-être mettre à part le cas exact cité ici ("Following receivers phone number are invalid") et continuer à alerter pour les autres.

#3

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Il faut arrêter d'utiliser cette vieille API, il y a une autre API beaucoup mieux.

#4

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

  • Description mis à jour (diff)
#5

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

Du coup on fait quoi ? On traite ce ticket quand même ?
On ferme ce ticket et on en ouvre un autre pour changer d'API ?

#6

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Lauréline Guerin a écrit :

Du coup on fait quoi ? On traite ce ticket quand même ?
On ferme ce ticket et on en ouvre un autre pour changer d'API ?

J'ai pas d'avis, si c'est pas très important on peut continuer à plâtrer cette implémentation; si on est certain qu'on voudra à terme bouger vers l'autre API (si c'est pas OVH qui déprécie avant) il me semble que passer plus de temps sur celle-ci est une perte de temps.

#7

Mis à jour par Thomas Noël il y a presque 4 ans

  • Statut changé de Nouveau à Rejeté
  • Priorité changé de Normal à Bas

Assez d'accord, laissons tomber ce http2sms vieillot et mal foutu. Regardons l'API normale https://docs.ovh.com/fr/sms/api_sms_cookbook/

Et en attendant, vivons avec ces alertes.

Formats disponibles : Atom PDF