in /modules/voiptwilio/includes/voiptwilioserver.inc

The integration uses the deprecated "Caller" and "Called" arguments with the 2010-04-01 Version of the API. These should be updated to use the supported "From" and "To" arguments.

https://www.twilio.com/docs/api/rest/making-calls

CommentFileSizeAuthor
#2 voipdrupal-fix-twilio-api-2263169-3.patch1.04 KBjohnstorey

Comments

johnstorey’s picture

johnstorey commented
Assigned: Unassigned » johnstorey

Unless there is objection, I'd like to work on this. We have a site that depends on this functionality. The 7.x-1.x branch is the current one I believe?

If I am unintentionally stepping on someone's feet here please let me know. It's been some time since I gave back to Drupal (about mid way through the 6.x lifecycle I believe) and I may have forgotten some proper procedures to follow.

johnstorey’s picture

johnstorey commented
StatusFileSize
new voipdrupal-fix-twilio-api-2263169-3.patch1.04 KB

Fix was simple -- just changing two api parameter names. I tested by writing a separate module to call the APIs and verify it all went through to my phone. Patch attached.

johnstorey’s picture

johnstorey commented
Status: Active » Needs review