[asterisk-dev] [Code Review] 4512: dns: Add res_resolver_unbound module with unit tests.

[Mark Michelson]

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


I notice that there are red blobs in the unit tests I wrote. I'll get those fixed up in the branch and post back here once I have that done.


/trunk/configs/samples/resolver_unbound.conf.sample
<https://reviewboard.asterisk.org/r/4512/#comment25330>

    I suggest putting a full path as the sample value here.



/trunk/res/res_resolver_unbound.c
<https://reviewboard.asterisk.org/r/4512/#comment25331>

    cfg is leaked here.



/trunk/res/res_resolver_unbound.c
<https://reviewboard.asterisk.org/r/4512/#comment25332>

    Should the default value for resolv be "system" ?


- Mark Michelson


On March 18, 2015, 1:13 p.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/4512/
> -----------------------------------------------------------
> 
> (Updated March 18, 2015, 1:13 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-24836
>     https://issues.asterisk.org/jira/browse/ASTERISK-24836
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This change adds a res_resolver_unbound module which uses libunbound and implements the core DNS resolver API. Queries can be started and cancelled as expected. Configuration also exists to change the behavior of the resolver some. Unit tests are present which use the libunbound zone and data API to add local records and then confirm they can be queried and are as expected.
> 
> 
> Diffs
> -----
> 
>   /trunk/res/res_resolver_unbound.c PRE-CREATION 
>   /trunk/makeopts.in 433107 
>   /trunk/configure.ac 433107 
>   /trunk/configs/samples/resolver_unbound.conf.sample PRE-CREATION 
>   /trunk/build_tools/menuselect-deps.in 433107 
> 
> Diff: https://reviewboard.asterisk.org/r/4512/diff/
> 
> 
> Testing
> -------
> 
> Hacked in a query to my own domain and confirmed it worked. Also ran the 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/20150318/f9d8e220/attachment-0001.html>