[Nagiosplug-devel] check_dns.c patch review request
Thomas Guyot-Sionnest
dermoth at aei.ca
Fri Jul 10 05:07:32 CEST 2009
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 08/07/09 05:25 PM, David Horn wrote:
> This is just a request to please review and/or commit the changes to
> check_dns to support IPv6 AAAA record types (and others).
>
> Let me know if there are any test cases you want me to write, or any
> additional information or changes needed to accept this patch.
>
> Patch is attached to this tracker entry:
> https://sourceforge.net/tracker/?func=detail&aid=2741269&group_id=29880&atid=397599
It looks good. I just have one question - on what basis did you increase
the ADDRESS_LENGTH macro? It would be helpful to know on what it's based
or how it was calculated.
Also, for inclusion we'll need to add tests too (plugins/t/check_dns.t).
If you can write them that would be awesome, otherwise it would really
help if you could at least give us a list of commands and what to expect
from them (return code and/or text match).
Thanks
- --
Thomas
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFKVrB06dZ+Kt5BchYRAjUkAKDRw7nHLrL/JDRkcxZAZmVm5+C8vgCeJqW4
/um/wrIiMznzBqBhv6R6B6A=
=Ovbo
-----END PGP SIGNATURE-----
More information about the Devel
mailing list