Skip to content

feat: Adding and improving logging calls to Stripe and Paypal controllers

stephen requested to merge app-logging into main

An earlier MR added baseline logging support to donate-neo but did not implement it as thoroughly as it might have - this MR seeks to address this by implementing logging where it might provide helpful insight or context to future maintainers.

This commit begins this process by (at last!) adding logging to the Stripe error handling method in the Stripe controller, and by adding logging to the exception handling in the payment vendor webhook handlers.

Logging calls in the Paypal controller were also reformatted away from the f-string string-formatting format (say that three times fast). This is because, when using the f-string formatting methodology, the string formatting is computed even when the logger will just throw the event away anyway. (See: https://docs.python.org/3/howto/logging.html#optimization)

Merge request reports