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

Unhandled webhooks can receive 200 responses, earlier logging and returning errors might help #31

Open
nicdwilson opened this issue Feb 13, 2025 · 2 comments
Milestone

Comments

@nicdwilson
Copy link
Collaborator

Describe the Bug

In several investigations, we have seen that merchants can show 200 responses to webhooks in their merchant dashboards, but these failed to create scheduled actions for asynchronous handling in Action Scheduler.

Event EV0676G554YSA2 which would have marked a submitted order as confirmed and moved the order from on-hold to processing is one such example.

Clearly this is difficult to reproduce, not neccesarily a bug and very dependent on the environment. Potentially though, logging the received webhook before attempting to set up the async handler, and possibly logging failures to create that handler, and returning error codes, will go a long way to making troubleshooting a little easier :)

@nicdwilson nicdwilson added the type: bug The issue is a confirmed bug. label Feb 13, 2025
@jeffpaul jeffpaul removed the type: bug The issue is a confirmed bug. label Feb 14, 2025
@sverleis
Copy link

Affected ticket(s): 9372251-zen

@jeffpaul jeffpaul added this to the 3.0.0 milestone Feb 25, 2025
@jeffpaul jeffpaul moved this from Triage to Backlog in 10up GC WooCommerce Project Feb 25, 2025
@nicdwilson
Copy link
Collaborator Author

Noting that an error response may prompt a webhook retry from GoCardless, which is what we want if the async handler is not successfully set up.

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

No branches or pull requests

3 participants