Merge pull request #13 from PetrDlouhy/report_status_change

Report status change
PetrDlouhy · Aug 16, 2024 · 770b2f1 · 770b2f1
2 parents a71798e + 53e2365
commit 770b2f1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/payments_payu/provider.py b/payments_payu/provider.py
@@ -597,6 +597,13 @@ def process_notification(self, payment, request):
                         type(payment).objects.filter(pk=payment.pk).update(
                             captured_amount=payment.captured_amount
                         )
+                    if status == PaymentStatus.CONFIRMED and payment.status != status:
+                        logger.warning(
+                            "Suspicious status change of payment %s: %s -> %s",
+                            payment.id,
+                            payment.status,
+                            status,
+                        )
                     payment.change_status(status)
                     return HttpResponse("ok", status=200)
         return HttpResponse("not ok", status=500)

diff --git a/tox.ini b/tox.ini
@@ -7,7 +7,7 @@ envlist =
 [testenv]
 setenv =
     PYTHONPATH = {toxinidir}:{toxinidir}/payments_payu
-commands = coverage run --source payments_payu runtests.py
+commands = python -W error::DeprecationWarning -m coverage run --source payments_payu runtests.py
 deps =
     django42: Django<4.3
     django50: Django<5.1