spookylukey / django-paypal

A pluggable Django application for integrating PayPal Payments Standard or Payments Pro
MIT License
729 stars 208 forks source link

Fix intermittent PDT issues #245

Closed djw closed 4 months ago

djw commented 3 years ago

Fixes issue #239.

Rather than use the same form for both the PDT callback and the postback, split these apart. We'll take a minimal set of parameters from the callback, and fill in rest using the postback endpoint.

I've included a test which includes the full set of query parameters I'm seeing on most (but not all) PDT callback requests. Note payment_date is in ISO format in these requests, and notify_version is a string. With this PR both these parameters will be ignored.

djw commented 4 months ago

I'm no longer using PDT, so can't vouch for the accuracy of this PR any more. Will close!