Development #21464
connecteurs SMS : utiliser self.requests
0%
Description
Pour que les choses soient logguées, qu'il y ait gestion timeout, etc.
Fichiers
Révisions associées
Historique
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-oxyd-use-self.requests-to-increase-logging-21464.patch 0001-oxyd-use-self.requests-to-increase-logging-21464.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
juste ça (ou bien je suis passé à côté) ?
Mis à jour par Frédéric Péters il y a plus de 4 ans
C'est ça mais pour tous les connecteurs SMS, pas juste Oxyd. (git grep SMSGatewayMixin)
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Fichier 0001-oxyd-use-self.requests-to-increase-logging-21464.patch 0001-oxyd-use-self.requests-to-increase-logging-21464.patch ajouté
- passerelle/apps/choosit
- passerelle/apps/mobyt
- passerelle/apps/orange (soap)
- passerelle/apps/ovh (déjà fait)
- passerelle/apps/oxyd
- passerelle/contrib/lille_urban_card (déjà fait)
Pour soap (passerelle/soap.py::Transport::send()
), on pourrait aussi utiliser self.request mais ce code n'est pas testé.
Mis à jour par Frédéric Péters il y a plus de 4 ans
passerelle/contrib/lille_urban_card (déjà fait)
Pas un connecteur SMS.
Mis à jour par Nicolas Roche il y a plus de 4 ans
oui, il y a un import inutilisé (peut-être en profiter pour le supprimer ?)
passerelle/contrib/lille_urban_card/models.py:
from passerelle.sms import SMSGatewayMixin
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Pour soap (passerelle/soap.py::Transport::send()), on pourrait aussi utiliser self.request mais ce code n'est pas testé.
Bon, ok pour pousser ainsi mais le connecteur SMS Orange, il est quand même plutôt utilisé, c'est dommage de passer à côté ainsi, voici donc #36921.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 1dee5d1c31e18522469e039113bb7891142506fa Author: Nicolas ROCHE <nroche@entrouvert.com> Date: Wed Oct 2 18:21:50 2019 +0200 sms: use self.requests to increase logging (#21464)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
sms: use self.requests to increase logging (#21464)