-
Notifications
You must be signed in to change notification settings - Fork 1
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
Throws an error when the request is a POST request. #44
Comments
@sam-lippert I just hit this issue when working with webhooks from Twilio and OpenPhone ... for external things like this we have no say, so we need to fix it. The logic in ctx.do should be to first read the body to a string with |
@nathanclevenger Can you provide a repro, please? |
@sam-lippert I just added you to the twill account - any POST web hook like this will cause this error ... Twilio let's you change webhooks to GET which is how I was able to get around this issue, but the default POST causes this crash https://console.twilio.com/us1/develop/phone-numbers/manage/incoming/PN4e29347f8fb098c25fdfa9453142a439/configure |
@AggressivelyMeows can you please provide @sam-lippert with a standalone minimalist repro |
https://github.com/drivly/airtable.do/blob/72641a4382ea369e5a63ef0033dc8bf84051324a/src/worker.js#L101
Has been updated to include the patch I used with webhooks.do. This tricks CTX.do into thinking its a GET request, so it doesnt try to parse the body.
The text was updated successfully, but these errors were encountered: