[asterisk-dev] [Code Review] 4528: dns: Add SRV recording parsing, sorting/weighting, and unit tests

Joshua Colp reviewboard at asterisk.org
Wed Apr 1 11:28:10 CDT 2015


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

(Updated April 1, 2015, 11:28 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 433889


Repository: Asterisk


Description
-------

This change adds the following:

1. As SRV records are added to a result the information is parsed and stored away in additional storage in the record. The SRV API can then be used to return this information.
2. Before invoking the DNS query callback the list of records on the result are sorted based on priority and weight.
3. Unit tests have been added which verify the record parsing, sorting, and weighting. There are also some off nominal which cover the cases when an invalid/corrupt record is received.
4. A unit test has also been added to res_resolver_unbound which adds an SRV record to a zone and confirms it is retrieved and parsed correctly.


Diffs
-----

  /trunk/tests/test_dns_srv.c PRE-CREATION 
  /trunk/res/res_resolver_unbound.c 433815 
  /trunk/main/dns_srv.c 433815 
  /trunk/main/dns_core.c 433815 
  /trunk/include/asterisk/dns_internal.h 433815 

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


Testing
-------

Executed unit tests and confirmed they pass.


Thanks,

Joshua Colp

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150401/9dc911af/attachment.html>


More information about the asterisk-dev mailing list