[PATCH] Fix server side DRSUAPI_DRS_GET_ANC handling (bug #12398)
metze at samba.org
Fri Dec 9 09:38:45 UTC 2016
>>> I think having more detailed tests and get the 100% exact behavior
>>> is desired, but a major effort.
>> I want to see shown that:
>> - additional objects are added to the reply
>> - that they are non-critical
>> - that they are added in a reasonable order (doens't need to be
>> exactly what windows does, but needs to be reasonable).
>> - In particular that a paternity sequence of critical -> non-
>> -> critical is handled correctly.
>> Replication bugs bite us pretty hard, as you have seen in my changes
>> here, I do want them very well tested.
> I know quite how busy you are, so after the great work he has done on
> DNS testing, I have asked Bob to make a start on the tests required
> here. It should be a good introduction to the guts of GetNCChanges ;-)
> I hope that helps move this task forward.
I've already started with it. See
This adds a test as source4/torture/drs/python/getnc.py
But I guess it should be somehow merged with
While the tests show that it's not 100% correct,
we're using the following patch in our packages
to avoid the problems with joining new servers:
Maybe Bob can go on from there.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: OpenPGP digital signature
More information about the samba-technical