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

Mark Michelson reviewboard at asterisk.org
Wed Mar 18 18:03:48 CDT 2015



> On March 18, 2015, 11:02 p.m., Mark Michelson wrote:
> > 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.

I've committed the whitespace fixes.


- Mark


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


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/b3d9b23a/attachment.html>


More information about the asterisk-dev mailing list