search

paypal / ipn-code-samples

Other
562 stars 486 forks source link

Update paypal_ipn.cfm #114

Closed csthsl closed 6 years ago

csthsl commented 6 years ago

Added a critical fix to the receiver_email. There is a disparity between an IPN simulation test and a real production IPN. Production IPN's encode the receiver_email. The simulator does not. If you decode the receiver_email, the validation will work in both environments.

overint commented 6 years ago

@pp-randy Are you aware of the difference between sandbox and live? Thanks for the PR!

randstraw commented 6 years ago

@overint yeah it was brought up here https://github.com/paypal/ipn-code-samples/issues/82. We can merge this in since the note should help if anyone has issues with this change.

csthsl commented 6 years ago

Thank you guys!

BigRedBot commented 6 years ago

Decoding the receiver email from the simulator that has a plus sign in it, will cause it to fail also. Perhaps they should make the simulator encode in the exact same way that live does, to prevent these kinds of confusions.