Development #43158
logs des SMS : ne pas logguer le message
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
20 mai 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Dans les SMS, on loggue trop d'info :
class SMSResource(BaseResource):
...
def send(self, request, *args, **kwargs):
...
logging.info('sending message %r to %r with sending number %r',
data['message'], data['to'], data['from'])
Il ne faut pas enregistrer le contenu du message, juste les metadonnées comme pour un mail.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 4 ans
- Fichier 0001-sms-do-not-log-message-content-43158.patch 0001-sms-do-not-log-message-content-43158.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Modifie bien les logs envoyées sur stderr (ou /var/log/passerelle)
$ passerelle-server ... sending SMS to ['+3369503699300'] from '+12015849510'
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 990ffe2b6388afb0c3e014a9c99c824d2032bb0a
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed May 20 15:02:27 2020 +0200
sms: do not log message content (#43158)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
sms: do not log message content (#43158)