Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Checks PayPal's returned business email against RECEIVER_EMAIL #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hugoatease
Copy link

PayPal allows users to have more than one email address attached to their account.

If such an email address is specified as RECEIVER_EMAIL, transaction's verification goes wrong because PayPal returns the primary account's email as receiver_email, but gives the right business address.

This small patch allows to check RECEIVER_EMAIL against returned receiver_email and business email if different.

@spookylukey
Copy link
Collaborator

Any chance of adding a test for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants