[Asterisk-code-review] pbx dundi: Added IPv6 support for dundi (asterisk[13])

[Adam Secombe]

Adam Secombe has posted comments on this change. ( https://gerrit.asterisk.org/6173 )

Change subject: pbx_dundi: Added IPv6 support for dundi
......................................................................


Patch Set 2:

We initially attempted to implement this change using netsock2, however we found that it would have required a significant amount more change to pbx_dundi.c in order to support it. To support it, we would have needed to change data structures and as such we decided that a less obtrusive change was to leave pbx_dundi with netsock instead of porting to netsock2. 

We have been using production systems with this patch since June of 2017 and it is very stable and as a result, we would like you to reconsider taking the patch as is. 

It adds quite a useful function to the asterisk baseline and as IPv6 becomes more and more commonly sought after, I am sure other end users will also be seeking this function.


-- 
To view, visit https://gerrit.asterisk.org/6173
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifbe298afc6416ba400db7be404a25994ad23968a
Gerrit-Change-Number: 6173
Gerrit-PatchSet: 2
Gerrit-Owner: Adam Secombe <adam.j.secombe at boeing.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Wed, 29 Nov 2017 05:34:38 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171128/d3f1fbd7/attachment.html>