-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4603/
-----------------------------------------------------------

Review request for Asterisk Developers.


Bugs: ASTERISK-24947
    https://issues.asterisk.org/jira/browse/ASTERISK-24947


Repository: Asterisk


Description
-------

This change adds the following:

1. A query set implementation. This is an API that allows queries to be 
executed in parallel and once all have completed a callback is invoked.
2. Unit tests for the query set implementation.
3. An external PJSIP resolver which uses the DNS core API to do NAPTR, SRV, 
AAAA, and A lookups.

For the resolver it will do NAPTR, SRV, and AAAA/A lookups in parallel. If 
NAPTR or SRV are available it will then do more queries. And so on. Preference 
is NAPTR > SRV > AAAA/A, with IPv6 preferred over IPv4. For transport it will 
prefer TLS > TCP > UDP if no explicit transport has been provided. Configured 
transports on the system are taken into account to eliminate resolved addresses 
which have no hope of completing.


Diffs
-----

  /trunk/tests/test_dns_query_set.c PRE-CREATION 
  /trunk/res/res_pjsip_session.c 434446 
  /trunk/res/res_pjsip/pjsip_resolver.c PRE-CREATION 
  /trunk/res/res_pjsip/include/res_pjsip_private.h 434446 
  /trunk/res/res_pjsip.c 434446 
  /trunk/main/dns_srv.c 434446 
  /trunk/main/dns_recurring.c 434446 
  /trunk/main/dns_query_set.c 434446 
  /trunk/main/dns_naptr.c 434446 
  /trunk/main/dns_core.c 434446 
  /trunk/include/asterisk/dns_query_set.h 434446 
  /trunk/include/asterisk/dns_internal.h 434446 
  /trunk/include/asterisk/dns_core.h 434446 
  /trunk/include/asterisk/autoconfig.h.in 434446 
  /trunk/configure.ac 434446 
  /configure UNKNOWN 

Diff: https://reviewboard.asterisk.org/r/4603/diff/


Testing
-------

Ran unit tests, they pass. Ran testsuite tests, they pass. Did spot checking 
using my own domains. They resolve as expected.


Thanks,

Joshua Colp

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to