Bug #21004
Signer les appels d'envoi de SMS
0%
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
Updated by Serghei Mihai almost 7 years ago
- File 0001-sms-sign-request-to-SMS-gateway-if-it-s-a-known-serv.patch 0001-sms-sign-request-to-SMS-gateway-if-it-s-a-known-serv.patch added
- Assignee set to Serghei Mihai
- Patch proposed changed from No to Yes
Updated by Thomas Noël over 6 years ago
- Status changed from Nouveau to En cours
À rebaser si possible, merci.
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
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
Updated by Serghei Mihai over 6 years ago
- File 0001-sms-sign-request-to-SMS-gateway-if-it-s-a-known-serv.patch 0001-sms-sign-request-to-SMS-gateway-if-it-s-a-known-serv.patch added
Rebasé et sans les sauts de ligne.
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.