Projet

Général

Profil

Bug #47034

envoi de sms : ?nostop n'est plus accepté

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

on reçoit une erreur « extra parameters: 'nostop' »


Fichiers

Révisions associées

Révision 6406ad80 (diff)
Ajouté par Thomas Noël il y a plus de 3 ans

sms: allow nostop parameter (#47034)

Révision 06ac4e4c (diff)
Ajouté par Thomas Noël il y a plus de 3 ans

sms: allow nostop parameter (#47034)

Historique

#1

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

C'est peut-être juste une typo (c'est noStop avec un S majuscule) https://docs.ovh.com/fr/sms/envoyer_des_sms_depuis_une_url_-_http2sms/ ou bien on s'est fait dénoncé (noStop ça veut dire : mon SMS n'est pas marketing merci de ne pas le limiter à la période 8h/20h en semaine, en plus du STOP XXXX).

Tu parles de notre API.

#2

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

C'est le connecteur.

-    @endpoint(perm='can_send_messages', methods=['post'])
-    def send(self, request, *args, **kwargs):
+    @endpoint(perm='can_send_messages', methods=['post'],
+              description=_('Send a SMS message'),
+              post={'request_body': {'schema': {'application/json': SEND_SCHEMA}}})
+    def send(self, request, post_data):

de #45829.

#3

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Fichier 0001-sms-allow-nostop-parameter-47034.patch ajouté
  • Tracker changé de Support à Bug
  • Statut changé de Nouveau à Solution proposée
  • Patch proposed changé de Non à Oui
#4

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Fichier 0001-sms-allow-nostop-parameter-47034.patch supprimé
#5

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

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

Ça doit passer en hotfix.

#6

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

  • Statut changé de Solution validée à En cours

(le patch n'était pas bon)

#7

Mis à jour par Thomas Noël il y a plus de 3 ans

Patch revu après auto-relecture... La seule présence de nostop dans la querystring (quelque soit la valeur) doit suffire à faire un stop=False, sinon on fait toujours stop=True.

#9

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

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

Mis à jour par Thomas Noël il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 06ac4e4c4040af714452cc3dac535e35dfb5c056
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Sep 25 15:57:25 2020 +0200

    sms: allow nostop parameter (#47034)

+ envoi en hotfix

#11

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