Default to nat=yes; warn when nat in general and peer differ
authorTerry Wilson <twilson@digium.com>
Mon, 21 Nov 2011 21:09:59 +0000 (21:09 +0000)
committerTerry Wilson <twilson@digium.com>
Mon, 21 Nov 2011 21:09:59 +0000 (21:09 +0000)
commit32d0faac9cb7576520d326ddd15d9fc7a0b47252
treef75414aa3d0e226e2cc130bf5f91e27c70a4d62c
parent298d015828a638a8ab0f07f8f8205ef04026bdd8
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
........

Merged revisions 345828 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 345830 from http://svn.asterisk.org/svn/asterisk/branches/10

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