]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Default to nat=yes; warn when nat in general and peer differ
authorTerry Wilson <twilson@digium.com>
Mon, 21 Nov 2011 20:53:47 +0000 (20:53 +0000)
committerTerry Wilson <twilson@digium.com>
Mon, 21 Nov 2011 20:53:47 +0000 (20:53 +0000)
commitda4f0afd6f772c4d087fc4e02af5b5ebdd8d281f
tree93b288d8a5f3b79886454b2859fb3512c594d29b
parentcfbd53182f8a60d12674de9b810a37b2e9780a48
Default to nat=yes; warn when nat in general and peer differ

It is possible to enumerate SIP usernames when the general and user/peer
nat settings differ in whether to respond to the port a request is sent
from or the port listed for responses in the Via header. In 1.4 and 1.6.2,
this would mean if one setting was nat=yes or nat=route and the other was
either nat=no or nat=never. In 1.8 and 10, this would mean when one was
nat=force_rport and the other was nat=no.

In order to address this problem, it was decided to switch the default
behavior to nat=yes/force_rport as it is the most commonly used option
and to strongly discourage setting nat per-peer/user when at all possible.

For more discussion of the issue, please see:
  http://lists.digium.com/pipermail/asterisk-dev/2011-November/052191.html

(closes issue ASTERISK-18862)
Review: https://reviewboard.asterisk.org/r/1591/
........

Merged revisions 345776 from http://svn.asterisk.org/svn/asterisk/branches/1.4
........

Merged revisions 345800 from http://svn.asterisk.org/svn/asterisk/branches/1.6.2

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@345828 65c4cc65-6c06-0410-ace0-fbb531ad65f3
CHANGES
channels/chan_sip.c
configs/sip.conf.sample