[asterisk-dev] [Code Review] 3781: Retrieve the source port of an incoming (chan_sip) SIP invite

[dtryba]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3781/
-----------------------------------------------------------

(Updated Aug. 6, 2014, 10:01 a.m.)


Review request for Asterisk Developers.


Changes
-------

Update to documentation in funcs/func_channel.c so "core show function CHANNEL" show the new option.


Bugs: ASTERISK-24040
    https://issues.asterisk.org/jira/browse/ASTERISK-24040


Repository: Asterisk


Description
-------

Retrieve the source port of an incoming (chan_sip) SIP invite in the dialplan with ${CHANNEL(recvport)}
To complement ${CHANNEL(recvip)} and enable me to make dialplan decisions based on source port (in a peerless setup, handle everything as guests using AGI to lookup source ip/port for routing/handling).

pjsip appears to have this capability through the CHANNEL function (pjsip,local_addr/remote_addr).

Simple 2 line patch using ast_sockaddr_stringify_fmt(const struct ast_sockaddr *sa, int format)
to return the port as a string.


Diffs
-----

  /trunk/channels/sip/dialplan_functions.c 418610 

Diff: https://reviewboard.asterisk.org/r/3781/diff/


Testing
-------

Tested on 11.10.2 (Debian Jessie) and trunk (418610) using IPv4. Having a few SIP endpoints connect from different address/ports combinations 
Logged ${CHANNEL(recvip)}:${CHANNEL(recvport)} corresponds with source ip:port in packetdumps on the asterisk machine.


File Attachments (updated)
----------------

func_channel.c.diff
  https://reviewboard.asterisk.org/media/uploaded/files/2014/08/06/a0056399-d2d1-4f7b-910a-f8bd37067b9d__func_channel.c.diff


Thanks,

dtryba

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140806/0d3b43b8/attachment.html>