[Home]

Summary:ASTERISK-15467: [patch] directmediapermit/directmediadeny support to restrict which peers can do directmedia based on ip address
Reporter:raarts (raarts)Labels:
Date Opened:2010-01-19 03:09:00.000-0600Date Closed:2010-05-20 12:54:03
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Channels/chan_sip/NewFeature
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) directmediapermit.patch
Description:Subject says it all. In some networks not all phones are fully routed, i.e.  not all phones can ping each other. This patch adds a way to restrict directmedia for certain peers between certain networks. Only implemented for the SIP channel.
Comments:By: Leif Madsen (lmadsen) 2010-01-19 07:44:05.000-0600

Thanks for the contribution! Any changes to chan_sip must go through a review process, so this is a candidate for https://reviewboard.asterisk.org. Thanks!
By: raarts (raarts) 2010-01-20 04:05:39.000-0600

Posted to https://reviewboard.asterisk.org/r/467/
By: raarts (raarts) 2010-01-25 05:35:22.000-0600

I bet it's written somewhere, but I can't find it: what should I do if the patch is reviewed, oej says it's ok, and I don't get any more comments?
By: Leif Madsen (lmadsen) 2010-01-25 08:46:31.000-0600

There isn't much to be done other than to wait for development resources to be available to review the patch and then commit the changes.
By: Digium Subversion (svnbot) 2010-05-20 12:54:02

Repository: asterisk
Revision: 264626

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

------------------------------------------------------------------------
r264626 | twilson | 2010-05-20 12:54:02 -0500 (Thu, 20 May 2010) | 15 lines

Add support for direct media ACLs

directmediapermit/directmediadeny support to restrict which peers can do
directmedia based on ip address. In some networks not all phones are fully
routed, i.e. not all phones can ping each other. This patch adds a way to
restrict directmedia for certain peers between certain networks.

(closes issue ASTERISK-15467)
Reported by: raarts
Patches:
     directmediapermit.patch uploaded by raarts (license 937)
Tested by: raarts

Review: https://reviewboard.asterisk.org/r/467/

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=264626