AccelerateNetworks / fusionpbx-webtexting

GNU Affero General Public License v3.0
0 stars 0 forks source link

[FEATURE] Delivery reports for outbound SMS/MMS messages #1

Open thefinn93 opened 1 year ago

thefinn93 commented 1 year ago

mod_sms seems to have a concept of delivery reports (via SMS::DELIVERY_REPORT events). We should send them after attempting to send a message, and add receiving handlers on the client side.

danry25 commented 1 year ago

Note that these are supposed to tie into the dlrid that sms.callpipe.com gets back from FirstPoint for outgoing SMS and MMSes and these provide a notification of whether the message was delivered to the network, or delivered to the handset itself (yet). On incoming SMS/MMS I am unclear if we should be providing a DLR response.

Digging now for further information on this, will update once found.

@uncheckederror Any input/info on passing the dlrid and other DLR data through to the client?

thefinn93 commented 1 year ago

this was just intended to cover cases where we POST to sms.callpipe.com and it doesnt give us 200 back. Delivery reports from higher up would also be nice to have.

uncheckederror commented 1 day ago

@dunkeltron what happens when a message from WebTexting is forwarded to sms.callpipe.com, but the API responds with an HTTP 50x or 40x? How does WebTexting handle non-200 responses for outbound messages?

dunkeltron commented 1 day ago

it looks like on line 76 of accelerate-networks.php we log any non 200 responses without returning the code to the vue app

dunkeltron commented 1 day ago

it looks like on line 76 of accelerate-networks.php we log any non 200 responses without returning the code to the vue app

uncheckederror commented 1 day ago

The goal here is to inform the WebTexting user that their outbound message failed to send.