Project

General

Profile

Bug #21004

Signer les appels d'envoi de SMS

Added by Frédéric Péters almost 7 years ago. Updated over 3 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
-
Start date:
04 January 2018
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
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.


Files

History

#2

Updated by Thomas Noël over 6 years ago

  • Status changed from Nouveau to En cours

À rebaser si possible, merci.

#4

Updated by Thomas Noël over 6 years ago

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

Updated by Anonymous over 6 years ago

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

Updated by Frédéric Péters over 6 years ago

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

Updated by Serghei Mihai over 3 years ago

  • Status changed from En cours to Fermé

Also available in: Atom PDF