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

Feature/paypal webhook verify #87

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Conversation

Jecko-o
Copy link
Contributor

@Jecko-o Jecko-o commented Apr 28, 2021

close #80

Jecko-o added 5 commits April 23, 2021 19:05
Add a `Sorter` service object whose role is to delegate the action to single event dedicated service object

Add a 'Base' class for the dedicated service objects to inherit from
This is needed to verify PayPal's webhooks signatures
Allow simpler API request building and refactor HTTPClient
@Jecko-o Jecko-o force-pushed the feature/paypal-webhook-verify branch from 78d7f73 to becfa86 Compare April 28, 2021 11:56
Jecko-o added 9 commits April 29, 2021 11:41
Set seller status to revoked on MERCHANT.PARTNER-CONSENT.REVOKED event.
Use `show_seller_status.payments_receivable` to update seller status

PayPal API required a new parameter to be added to the extension config
This prevent bugs related to access token expiration.

Another approch could be resetting it at every execution.
Every call to the paypal webhooks endpoint must be validated to proceed
@Jecko-o Jecko-o force-pushed the feature/paypal-webhook-verify branch from becfa86 to df5b367 Compare April 29, 2021 10:44
@stale
Copy link

stale bot commented Jul 23, 2021

This issue has been automatically marked as stale because it has not had recent activity. It might be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Webhook verification doesn't work
1 participant