[Freeipa-devel] [PATCH] 0059-0063 Update DNSSEC attributes/record types
Martin Basti
mbasti at redhat.com
Fri Jun 13 07:55:45 UTC 2014
On Thu, 2014-06-12 at 16:20 +0200, Martin Basti wrote:
> On Thu, 2014-06-12 at 13:17 +0200, Petr Vobornik wrote:
> > On 9.6.2014 17:28, Martin Basti wrote:
> > > Ticket: https://fedorahosted.org/freeipa/ticket/4328
> > >
> > > Petr please make the WebUI patch review (0062) :-)
> > >
> > > Patches attached.
> > >
> >
> > Patch #0059: LGTM
> >
> > Patch #0060:
> >
> > 1. Please add `pattern_errmsg` to `salt` part of nsec3param. Otherwise
> > you get general "Text does not match field pattern" error message in Web UI.
> >
> I will add the message.
>
> > 2. Could be in one if:
> > + if nsec3params is not None:
> > + if len(nsec3params) > 1:
> >
> > Maybe I'm missing something. But why does the dns plugin code use
> > following all over the place:
> >
> > try:
> > nsec3params = rrattrs['nsec3paramrecord']
> > except KeyError:
> > pass
> > else:
> > if nsec3params is not None:
> >
> > instead of:
> >
> > nsec3params = rrattrs.get('nsec3paramrecord')
> > if nsec3params is not None:
> >
> > btw you use both patterns in the patch.
> I will use shorten form, I wrote it in the same way as the other code in
> the block was written, that's why.
>
> >
> > Patch #0061: ACK
> >
> >
> > Patch #0062:
> >
> > 3. Why are there the idnafsdbrec1 variables?
> >
> It was replace for NSEC records, which are not supported anymore.
> Now I realize, there is wrong description, so the idnafsbrec1 variable
> is not needed.
> I will remove it.
>
> > 4. related to ^^:
> > ./ipatests/test_xmlrpc/test_dns_plugin.py:199:33: E231 missing
> > whitespace after ','
> >
> >
> > Patch #0063: LGTM
> > IDK if they work because I'm experiencing some weird issues with
> > xmlrpc_tests in general.
> I had troubles only with permission tests, but all DNS test worked fine for me.
>
> Thank you for review.
>
Updated patches attached.
--
Martin^2 Basti
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0059-2-DNSSEC-remove-unsuported-records.patch
Type: text/x-patch
Size: 22390 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140613/3e5688d9/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0060-2-DNSSEC-added-NSEC3PARAM-record-type.patch
Type: text/x-patch
Size: 31803 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140613/3e5688d9/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0061-2-DNSSEC-webui-update-DNSSEC-attributes.patch
Type: text/x-patch
Size: 6106 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140613/3e5688d9/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0062-2-Tests-remove-unused-records-from-tests.patch
Type: text/x-patch
Size: 4534 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140613/3e5688d9/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0063-2-Tests-tests-for-NSEC3PARAM-records.patch
Type: text/x-patch
Size: 3186 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140613/3e5688d9/attachment-0004.bin>
More information about the Freeipa-devel
mailing list