[Freeipa-devel] [PATCH 0184] Always return absolute idnsname in dnszone commands
Martin Basti
mbasti at redhat.com
Thu Jan 15 13:58:25 UTC 2015
On 15/01/15 14:25, Jan Cholasta wrote:
> Hi,
>
> Dne 15.1.2015 v 13:27 Martin Basti napsal(a):
>> On 15/01/15 13:17, Martin Basti wrote:
>>> https://fedorahosted.org/freeipa/ticket/4722
>>>
>>> Patch attached.
>>>
>> Fast fix.
>>
>> Updated patch attached.
>
> 1) Forward zone commands are not fixed.
FWzones are new and always normalized to absolute name in ldap
>
> 2) It seems that the primary key returned by -mod, -del and -show
> (.result.value) is made absolute somewhere else in the code. Would it
> be possible to do it in one place?
IMO it is not possible.
Value is generated from key, and key is normalized to absolute zone
before calling execute.
LDAPUpdate:
...
if self.obj.primary_key:
pkey = keys[-1]
else:
pkey = None
return dict(result=entry_attrs, value=pkey_to_value(pkey, options))
The idnsname attribute is just taken from LDAP without any normalization
>
> 3) Attribute values returned from LDAP are never None, so the if
> should be "if 'idnsname' in entry_attrs:".
Ok I will revert the change I made.
>
> 4) If idnsname always has only single value, use
> "entry_attrs.single_value['idnsname'] =
> entry_attrs.single_value['idnsname'].make_absolute()"
Thanks
>
> Honza
>
Updated patch attached.
--
Martin Basti
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0184.3-Always-return-absolute-idnsname-in-dnszone-commands.patch
Type: text/x-patch
Size: 2153 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150115/b23af154/attachment.bin>
More information about the Freeipa-devel
mailing list