Bug #19933
ne pas annoncer qu'une transaction est inconnue alors qu'on l'a trouvée
Début:
07 novembre 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
try:
transaction = Transaction.objects.get(order_id=payment_response.order_id)
except Transaction.DoesNotExist:
logger.warning(u'received unknown payment response with id %s',
smart_text(payment_response.order_id), extra=extra_info)
raise Http404
else:
extra_info['lingo_transaction_id'] = transaction.pk
if transaction.user:
# let hobo logger filter handle the extraction of user's infos
extra_info['user'] = transaction.user
logger.warning(u'received unknown payment response with id %s',
smart_text(payment_response.order_id), extra=extra_info)
Le deuxième logger.warning devrait sans doute plutôt être logger.info, d'autre part et surtout, il devrait contenir "known payment response" et pas "unknown payment response".
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-lingo-fix-logging-of-known-payment-response-19933.patch 0001-lingo-fix-logging-of-known-payment-response-19933.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 6 ans
Je compléterais encore, mais je sais pas si c'est vraiment utile d'avoir le transaction.pk directement dans al ligne de log
logger.info(u'received payment response with id %s for transaction %s',
smart_text(payment_response.order_id), transaction.pk, extra=extra_info)
Sinon ack ainsi, peut-être en retirant quand même le "known" qui me semble un peu évident.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Selon moi pas utile, l'order_id est déjà unique.
commit 79992303d62beacb33fc845423f74e28cc8f41ed
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Nov 7 09:42:04 2017 +0100
lingo: fix logging of known payment response (#19933)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: fix logging of known payment response (#19933)