[Asterisk-code-review] res_pjsip_messaging: Refactor outgoing URI processing (asterisk[16])

George Joseph asteriskteam at digium.com
Thu May 27 07:35:17 CDT 2021


Hello Joshua Colp, Kevin Harwell, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/15828

to look at the new patch set (#4).

Change subject: res_pjsip_messaging: Refactor outgoing URI processing
......................................................................

res_pjsip_messaging: Refactor outgoing URI processing

 * Implemented the new "to" parameter of the MessageSend()
   dialplan application.  This allows a user to specify
   a complete SIP "To" header separate from the Request URI.

 * Completely refactored the get_outbound_endpoint() function
   to actually handle all the destination combinations that
   we advertized as supporting.

 * We now also accept a destination in the same format
   as Dial()...  PJSIP/number at endpoint

 * Added lots of debugging.

ASTERISK-29404
Reported by Brian J. Murrell

Change-Id: I67a485196d9199916468f7f98bfb9a0b993a4cce
---
A doc/CHANGES-staging/res_pjsip_messaging.txt
M res/res_pjsip_messaging.c
2 files changed, 693 insertions(+), 101 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/28/15828/4
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15828
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I67a485196d9199916468f7f98bfb9a0b993a4cce
Gerrit-Change-Number: 15828
Gerrit-PatchSet: 4
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210527/9ed5c0b5/attachment-0001.html>


More information about the asterisk-code-review mailing list