[PATCH] Fix dn+binary linked attribute behavior (bug 11139)
Bob Campbell
bobcampbell at catalyst.net.nz
Thu Jan 5 20:44:58 UTC 2017
Hi Douglas,
Thanks for having a look at this.
Attached are patches with these issues fixed.
Thanks,
Bob
On 05/01/17 17:05, Douglas Bagnall wrote:
> hi Bob,
>
> I see a couple of superficial issues.
>>
>> +import os
> You don't actually use this.
>
>> @@ -37,12 +40,11 @@ from ldb import (
>> Message,
>> FLAG_MOD_ADD,
>> FLAG_MOD_REPLACE,
>> + FLAG_MOD_DELETE
>> )
> or this.
>
>> +
>> + link_end_dn = ldb.Dn(self.ldb_dc1, "OU=test_ou,%s" % str(self.ldb_dc1.get_default_basedn()))
> here the line length is getting a bit out of control and the str() is
> redundant (because "%s" implies str()). There are other instances of that.
>
>> +
>> + self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, nc_dn=self.schema_dn, forced=True)
>> + self._net_drs_replicate(DC=self.dnsname_dc2, fromDC=self.dnsname_dc1, nc_dn=self.domain_dn, forced=True)
>> +
> and line length again.
>
>> From 0a1a65c99e6b8ec39ad0c7018e05759343899e51 Mon Sep 17 00:00:00 2001
>> From: Andrew Bartlett <abartlet at samba.org>
>> Date: Mon, 12 Oct 2015 15:51:37 +1300
>> Subject: [PATCH 2/2] repl_meta_data: Remove the correct linked attribute
>> forward link
> This one looks good.
>
> cheers,
> Douglas
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-torture-drs-Add-a-test-for-dn-binary-linked-attribut.patch
Type: text/x-patch
Size: 8214 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170106/3a5444c6/0001-torture-drs-Add-a-test-for-dn-binary-linked-attribut.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-repl_meta_data-Remove-the-correct-linked-attribute-f.patch
Type: text/x-patch
Size: 6839 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170106/3a5444c6/0002-repl_meta_data-Remove-the-correct-linked-attribute-f.bin>
More information about the samba-technical
mailing list