[asterisk-dev] Re: [asterisk-commits] trunk r9294 - in /trunk: channels/chan_sip.c configs/sip.conf.sample

[Kevin P. Fleming]

asterisk-commits at lists.digium.com wrote:
> Author: oej
> Date: Thu Feb  9 09:40:53 2006
> New Revision: 9294
> 
> URL: http://svn.digium.com/view/asterisk?rev=9294&view=rev
> Log:
> - Change "rfc2833" to "rtp" in sip.conf. Keeping backwards compatibility.

Vetoed.

This change is wrong; there are other RTP formats for DTMF events
besides RFC2833 (Cisco DTMF is one), and to be pedantic 'inband' DTMF is
also in the RTP stream.

What is the motivation for this change? Are we also going to change the
nat=RFC<foo> setting to something else?