<p>Adam Secombe <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/6173">View Change</a></p><p>Patch set 3:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Joshua, </p><p style="white-space: pre-wrap; word-wrap: break-word;">In order to use netsock2, we would have had to change the address structure to ast_sockaddr but as the original dundi implementation that used ipv4 did not adopt it we decided not to change it. We were trying to minimize the amount of code change, whilst adding the ipv6 functionality to minimise the chance of introducing new bugs to dundi.</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for your original two questions:<br>Q) "Is it possible for an IPv4 host to receive an IPv6 address? If so for older versions what occurs?"</p><p style="white-space: pre-wrap; word-wrap: break-word;">A) We copied the way IAX2 implemented the IPv6 interface - in this scenario we would not expect two Asterisk servers that are configured to talk together with DUNDi to be running a mixed fleet of IPv4 and IPv6, you would simply choose one and make sure your entire fleet of Asterisk servers are running the same version - as a result we don't see any issue with it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Q) "I don't think this works with IPv6 due to the scanning for : which is present in IPv6 addresses"</p><p style="white-space: pre-wrap; word-wrap: break-word;">A) We have not seen any issues with this implementation in more than 6 months of trials with a significant number of servers. We will remove the TODO statement as we beleive is not needed. </p><p style="white-space: pre-wrap; word-wrap: break-word;">Have submitted a new patch set that removes this TODO statement.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Kind regards,</p><p style="white-space: pre-wrap; word-wrap: break-word;">Adam</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/6173">change 6173</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6173"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ifbe298afc6416ba400db7be404a25994ad23968a </div>
<div style="display:none"> Gerrit-Change-Number: 6173 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Adam Secombe <adam.j.secombe@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: Adam Secombe <adam.j.secombe@boeing.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 04 Dec 2017 04:44:45 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>