Skip to content
This repository has been archived by the owner on May 31, 2019. It is now read-only.

withWebhook middleware seems incompatible with /customers/data_request mandatory webook #135

Closed
oortlieb opened this issue Sep 20, 2018 · 1 comment

Comments

@oortlieb
Copy link

I'm currently working on the mandatory webooks for my Shopify app. It looks like the withWebook middleware provides a response prior to forwarding the information to the next handler. For the erasure endpoints, this probably isn't a huge issue, because that will be a deferred job on our end anyway. What is the suggested way to implement the handler for the /customers/data_request webhook? It seems that the customer's data is expected to provided by the response to the hook.

@oortlieb
Copy link
Author

After reading the docs a bit closer, it looks like I was incorrect about needing to provide the customer data to the request. In this case, the 200 response from the withWebhook handler should be fine. The endpoint just needs acknowledge the request so that a data collection process can be executed asynchronously.

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

No branches or pull requests

1 participant