Projet

Général

Profil

Development #46316

Connecteur SMS OVH : avoir le crédit restant affiché dans le connecteur

Ajouté par Marie Kuntz -> retour le 13 mai il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Afin d'avoir accès au crédit restant directement dans le connecteur sans avoir à faire des calculs manuels à base de "on a prévu X et on a envoyé Y" sachant qu'on récupère Y sur l'interface d'OVH en additionnant des chiffres dont on n'est pas sûrs qu'ils soient corrects


Fichiers

0001-ovh-update-credit-left-42921.patch (10,7 ko) 0001-ovh-update-credit-left-42921.patch Valentin Deniaud, 08 octobre 2020 11:44

Demandes liées

Lié à Passerelle - Development #42921: connecteurs SMS, possibilité d'alerte au "niveau d’alerte du crédit d’appel"Fermé15 mai 2020

Actions

Révisions associées

Révision 153a9137 (diff)
Ajouté par Valentin Deniaud il y a plus de 3 ans

ovh: update credit left (#46316)

Historique

#3

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Lié à Development #42921: connecteurs SMS, possibilité d'alerte au "niveau d’alerte du crédit d’appel" ajouté
#4

Mis à jour par Valentin Deniaud il y a plus de 3 ans

Je pensais qu'on allait pouvoir récupérer le nombre de sms envoyés par utilisateur via l'API nouvellement prise en charge, mais ce n'est pas le cas...

Le connecteur expose aussi un champ « credit_left » qui n'est à mon avis jamais regardé, puisqu'il correspond au nombre de crédit restant sur le compte sms, rien à voir avec l'utilisateur API qui nous intéresse.

Il me faudrait être sûr de ce qui est attendu au niveau fonctionnel : ma compréhension c'est qu'on vend au client x sms, et qu'il faut lui demander d'en racheter lorsque ces sms ont été envoyés. D'où le besoin de connaître le nombre de sms envoyés à tout moment, et éventuellement via le ticket lié d'avoir une alerte quand il ne reste que 100 sms, par ex.

Mais on ne peut pas faire confiance à passerelle pour maintenir un tel nombre, les clients ayant accès à /admin/ pourraient le changer comme bon leur semble, même si en pratique ça n'arrive pas c'est pas fou. En mode plus innocent, dupliquer le connecteur permettrait d'avoir deux fois plus de sms, bref c'est niet.

J'ai tout de même une idée pour avoir un truc qui marche :
  • sur l'utilisateur api d'un client, mettre un quota correspondant aux sms achetés (ne pas toucher à limite qui ne sert à rien).
  • il est alors possible de récupérer via l'API le quota restant, de l'afficher, et de configurer une alerte quand il est trop bas.

C'est un peu le miroir du nombre de SMS envoyés, est-ce que ça pourrait convenir ? Bémol, d'après la doc OVH quand le quota restant tombe à zéro l'envoi est bloqué, il faudra donc agir vite à la réception de l'alerte pour remettre des pièces dans la machine, ou mettre un seuil d'alerte assez haut.

#9

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Assigné à mis à Valentin Deniaud
#11

Mis à jour par Valentin Deniaud il y a plus de 3 ans

Trucs à noter :

  • Plus de support des utilisateurs d'API (vérifié en recette et en prod que personne n'avait commencé à jouer avec ça) :
    -    API_URL = 'https://eu.api.ovh.com/1.0/sms/%(serviceName)s/users/%(login)s/jobs/'
    +    API_URL = 'https://eu.api.ovh.com/1.0/sms/%(serviceName)s/'
    
  • Au détour de #45814 il y a déjà une ligne « Crédit restant » affichée dans le manager. Sauf qu'avec l'ancienne API, elle correspond aux crédits restants sur le compte, alors que le connecteur utilise un utilisateur d'API, ça n'apporte donc que de la confusion : je masque l'info dans ce cas.

Voilà sinon une bonne partie du patch c'est de déplacer le code de requête/signature dans une méthode à part, et aussi pour ne pas se retrouver avec le mécanisme base_adresse « attendez une heure pour que l'info soit synchronisée », j'ajoute un job lors d'un save().

Derrière ce ticket il va y avoir tout de suite #42921, pour envoyer une alerte quand le crédit est bas.

#12

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

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

Mis à jour par Valentin Deniaud il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 171e08e509b42dc57fb8d33ec054d8b86e9887e4
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Wed Oct 7 11:22:46 2020 +0200

    ovh: update credit left (#46316)
#14

Mis à jour par Frédéric Péters il y a plus de 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF