----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4528/ -----------------------------------------------------------
(Updated March 31, 2015, 3:50 p.m.) Review request for Asterisk Developers. 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 (updated) ----- /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
-- _____________________________________________________________________ -- 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
