<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On 19 Mar 2014, at 11:27, Sean Bright <<a href="mailto:reviewboard@asterisk.org">reviewboard@asterisk.org</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><blockquote style="font-family: Verdana, Arial, Helvetica, sans-serif; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; margin-left: 1em; border-left-width: 2px; border-left-style: solid; border-left-color: rgb(208, 208, 208); padding-left: 10px;"><blockquote style="margin-left: 1em; border-left-width: 2px; border-left-style: solid; border-left-color: rgb(208, 208, 208); padding-left: 10px;"><pre style="white-space: pre-wrap; word-wrap: break-word;">I hadn't considered multihomed machines when I wrote this patch, so that might throw a wrench in this.</pre></blockquote></blockquote><pre style="font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: pre-wrap; word-wrap: break-word;">There are a log of things wrong with this. Going to re-think it some.</pre><br class="Apple-interchange-newline"></blockquote></div>The major one being that ICE was create so that we don't have to mess around like this. The external IP is just a server-reflexive address. You may want to make a decision on what you prefer in the C-line depending on the old logic - what's the most likely address we're going to end up with?<div><br></div><div>/O</div></body></html>