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.
Fichiers
Historique
Mis à jour par Serghei Mihai il y a plus de 6 ans
- Fichier 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 ajouté
- Assigné à mis à Serghei Mihai
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 6 ans
- Statut changé de Nouveau à En cours
À rebaser si possible, merci.
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 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 ajouté
Rebasé.
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
Mis à jour par Serghei Mihai il y a environ 6 ans
- Fichier 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 ajouté
Ok.
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
Mis à jour par Serghei Mihai il y a presque 6 ans
- Fichier 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 ajouté
Rebasé et sans les sauts de ligne.
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.