claudiajs / claudia-bot-builder

Create chat bots for Facebook Messenger, Slack, Amazon Alexa, Skype, Telegram, Viber, Line, GroupMe, Kik and Twilio and deploy to AWS Lambda in minutes
https://claudiajs.com/claudia-bot-builder.html
MIT License
1.84k stars 252 forks source link

Fixes Twilio responses. #44

Closed philnash closed 7 years ago

philnash commented 7 years ago

Twilio expects an XML response to an incoming webhook, if it gets a content type like application/json it will show errors in the Twilio debugger. This commit ensures that the bot responds with a content type of text/xml and the an empty element in the body.

simalexan commented 7 years ago

@philnash thanks for the effort! 👍

philnash commented 7 years ago

Would it be possible to cut a release with this fix? Anyone using claudia-bot-builder with Twilio will just be seeing errors in their Twilio console at the moment? I've also written a blog post on how to use claudia-bot-builder for the Twilio blog, but don't want to publish if it will cause errors.

Thanks!

simalexan commented 7 years ago

@philnash yeah, we shall make a release today. Was planned already

philnash commented 7 years ago

Sweet, thanks!

simalexan commented 7 years ago

@philnash you are welcome!

stojanovic commented 7 years ago

It's published https://github.com/claudiajs/claudia-bot-builder/releases/tag/v2.4.1