fix wrong nsDS5ReplicaHost on replication agreements for more then two hosts

Authored by jankow on Dec 10 2017, 1:03 AM.



this should fix T3283

Test Plan

set up replication with three or more ldap-servers on base domain

add another domain. They should have proper replication agreements.

Diff Detail

rPNL php-Net_LDAP3
Lint Skipped
Unit Tests Skipped

Event Timeline

jankow created this revision.Dec 10 2017, 1:03 AM
pasik added a subscriber: pasik.Dec 23 2017, 12:04 PM
machniak edited reviewers, added: Net_LDAP3 Developers; removed: Net_LDAP3.May 18 2018, 2:31 PM

I just wanted to ask if someone already took a look at this patch. T3283 is still affecting replication management.

Actually I don't understand really how it work's in fabricator that people get notified. Since the related bug report is closed I don't know if anybody will ever have a look here...

anyone who could review this?

machniak abandoned this revision.Fri, Jun 2, 3:41 PM
machniak added a subscriber: machniak.

Improved fix in D4352.