Projet

Général

Profil

Bug #21004

Signer les appels d'envoi de SMS

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
04 janvier 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Aujourd'hui,

            response = requests.post(settings.SMS_GATEWAY_URL, json=data, proxies=settings.REQUESTS_PROXIES)

fait que l'envoi d'SMS est tout théorique, ou demande une ouverture particulière de Passerelle.


Fichiers

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 6 ans

#2

Mis à jour par Thomas Noël il y a environ 6 ans

  • Statut changé de Nouveau à En cours

À rebaser si possible, merci.

#4

Mis à jour par Thomas Noël il y a environ 6 ans

En cas d'erreur qui ne renvoie pas du JSON on va salement crasher ici non ? (je n'aurais pas supprimé cette ligne) :

...
         try:
             response = requests.post(settings.SMS_GATEWAY_URL, json=data, proxies=settings.REQUESTS_PROXIES)
-            response.raise_for_status()
             if not response.json()['err']:
...

Et sinon j'ai l'impression que c'est n'importe quoi, ici :

from requests import Response, Session as RequestsSession, RequestException

Garde plutôt le import requests et utilise requests.Session et requests.Exception

#6

Mis à jour par Anonyme il y a presque 6 ans

Serghei Mihai a écrit :

Ok.

Est-ce tu pourrais rebaser sur le master stp ?
Avant de tester, je remarque déjà des sauts de lignes en triple, en début de code

#8

Mis à jour par Frédéric Péters il y a presque 6 ans

Il y a l'import de requests du coup l'import explicite de l'exception et la modification au bout de code l'utilisant me semblent à virer.

#9

Mis à jour par Serghei Mihai il y a presque 3 ans

  • Statut changé de En cours à Fermé

Formats disponibles : Atom PDF