-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[AC-1705] Update PayPal Verification URL to point to updated domain #3338
[AC-1705] Update PayPal Verification URL to point to updated domain #3338
Conversation
New Issues
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
URLs are valid per https://developer.paypal.com/api/nvp-soap/ipn/IPNImplementation/.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding logging! If you wouldn't mind, please update to include some descriptive messages in the logs. I left some comments with my suggestions, but ultimately it's up to you
…point-to-updated-domain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you sir!
…point-to-updated-domain
…point-to-updated-domain
…point-to-updated-domain
…point-to-updated-domain
Type of change
Objective
PayPal updated its IPN verification callback URL to include a subdomain https://ipnpb.sandbox.paypal.com/cgi-bin/webscr (for Sandbox IPNs) and https://ipnpb.paypal.com/cgi-bin/webscr (for live IPNs)
Code changes
Before you submit
dotnet format --verify-no-changes
) (required)