django-payments icon indicating copy to clipboard operation
django-payments copied to clipboard

Published 20 hours ago • verified publisher icon jazzband

Improved Logging

Open jonathanmeier5 opened this issue 7 years ago • 1 comments

I've recently implemented PayPal transactions with django-payments. There does not appear to be any logging for failed requests to the PayPal API, and instead the app just displays an "Internal Paypal Error" message without any details about the response. If I added logging to write some part of the HTTP response would you be open to accepting the pull request?

jonathanmeier5 avatar Jul 28 '17 23:07 jonathanmeier5

A PR with improve logging is most welcome.

As a rule though:

  • ERROR should be stuff like an invalid API response for paypal or alike.
  • INFO should be stuff like "payment success", "payment rejected".
  • DEBUG can include API responses and finer grained stuff that you'd only want while debugging/testing.

WhyNotHugo avatar Oct 10 '20 12:10 WhyNotHugo