[asterisk-dev] [Code Review] SIP: peer matching by callbackextension

David Vossel reviewboard at asterisk.org
Thu Dec 9 18:16:43 UTC 2010

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-12-09 12:16:42.954704)

Review request for Asterisk Developers.


This update is bas's port of this patch to 1.8 and trunk.  It adds an option to turn on and off matching by callbackextension.


If there are a number of peers with different callbackextension parameters and the same host address.  The first peer found matching the address is used regardless if that peer's callbackextension matches the incoming extension or not.

Now, to better match peers with incoming calls, if an incoming call's address can match multiple peers by address, we check each of those peer's callbackextension against the incoming extension for the best possible match.

It is possible that my implementation may be too expensive and only serve to address a minor edge case in the usage of chan_sip.  I do not fully understand the impact my changes may have upon performance when a large number of peers are present.  This patch assumes the new parse_uri() change has been made.

for example with two peers as follows

incoming calls to 9991 and to 9992 are both matched to the peer trunk1

This addresses bug 14340.

Diffs (updated)

  /trunk/channels/chan_sip.c 297950 
  /trunk/channels/sip/include/sip.h 297950 
  /trunk/configs/sip.conf.sample 297950 

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


Tested multiple peers with the same address containing different callbackextensions. Verified the correct peers were matched with incoming calls.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20101209/f1ae22fb/attachment.htm 

More information about the asterisk-dev mailing list