<html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body text="#000000" bgcolor="#FFFFFF"> On 10/19/2011 08:15 AM, Martin Kosek wrote: <blockquote cite="mid:1319026515.9647.13.camel@balmora.brq.redhat.com" type="cite"> <pre wrap="">On Wed, 2011-09-07 at 15:18 +0200, Martin Kosek wrote: </pre> <blockquote type="cite"> <pre wrap="">On Wed, 2011-09-07 at 15:05 +0200, Martin Kosek wrote: </pre> <blockquote type="cite"> <pre wrap="">This is 3.0 Core Effort Backlog patch. The changes to API may look scary, but it should be OK, I just added validators and normalizers. I found a lot of RR types unsupported by bind-dyndb-ldap. I implemented a validator telling this information to the user. I think the message is more user-friendly than the previous LDAP schema error. Enjoy the RFCs! :-) Martin --- Implement missing validators for DNS RR types so that we can capture at least basic user errors. Additionally, a normalizer creating a fully-qualified domain name has been implemented for several RRs to prevent this common user error. <a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/1106">https://fedorahosted.org/freeipa/ticket/1106</a> </pre> </blockquote> <pre wrap=""> I noticed a typo in format description for LOC record validation. A fixed patch attached. Martin </pre> </blockquote> <pre wrap=""> Rebased for current master. This patch is still waiting for review. As I would like to base my next DNS work (structured DNS commands) on this patch I would like to have it reviewed soon. Thanks, Martin </pre> <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ Freeipa-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre> </blockquote> I've just given it a visual review, but it looks right. Probably should have some unit tests to go with it for some of the more commonly used types. </body> </html>