search

389ds / 389-ds-base

The enterprise-class Open Source LDAP server for Linux
https://www.port389.org/
Other
211 stars 91 forks source link

cannot set repl referrals or state #277

Closed 389-ds-bot closed 4 years ago

389-ds-bot commented 4 years ago

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/277

Setting up replication leads to many errors like this: [01/Feb/2012:20:56:00 -0700] NSMMReplicationPlugin - repl_set_mtn_referrals: could not set referrals for replica dc=example,dc=com: 32

389-ds-bot commented 4 years ago

Comment from rmeggins (@richm) at 2012-02-03 05:03:20

0001-Ticket-277-cannot-set-repl-referrals-or-state.patch 0001-Ticket-277-cannot-set-repl-referrals-or-state.patch

389-ds-bot commented 4 years ago

Comment from rmeggins (@richm) at 2012-02-03 05:17:04

To ssh://git.fedorahosted.org/git/389/ds.git 2a0b622..bbb4139 master -> master commit changeset:bbb41398a57f6ac1daf9828943768e720cc70245/389-ds-base Author: Rich Megginson richm@redhat.com Date: Thu Feb 2 09:38:49 2012 -0700 Reviewed by: nkinder (Thanks!) Branch: master Fix Description: Need to create a normalized DN string using slapi_create_dn_string rather than a raw unnormalized DN string with slapi_ch_smprintf. The code was using the unnorm string to perform the internal search and modify operations, and the modify code was expecting a normalized DN string instead. The search code was normalizing the DN, but the comment in the function suggested that it was supposed to return a normalized DN string. Platforms tested: RHEL6 x86_64 Flag Day: no Doc impact: no

389-ds-bot commented 4 years ago

Comment from nhosoi (@nhosoi) at 2012-02-03 05:19:46

The bug was introduced by this patch. Sorry about it... :( commit f6397113666f06848412bb12f754f04258cfa5fa Author: Noriko Hosoi nhosoi@redhat.com Date: Wed Oct 19 16:51:48 2011 -0700 Reduce the number of DN normalization

In your patch, you are calling "slapi_sdn_get_ndn(root)" and slapi_sdn_get_dn(root), both may normalize root if the ndn or dn value is not available yet, respectively. Then, slapi_create_dn_string normalizes it again. So, could you replace slapi_sdnget(n)dn with slapi_sdn_get_udn? It'd eliminate one dn normalization each... 2912 dn = slapi_create_dn_string("cn=\"%s\",%s",
2913 slapi_sdn_get_ndn(root), MAPPING_TREE_BASE_DN); 

2939        dn = slapi_create_dn_string("cn=\"%s\",%s",  
2940                                    slapi_sdn_get_dn(root), MAPPING_TREE_BASE_DN);
389-ds-bot commented 4 years ago

Comment from rmeggins (@richm) at 2012-02-03 05:24:11

To ssh://git.fedorahosted.org/git/389/ds.git b4c3983..43ec9da master -> master commit changeset:43ec9daa67ff67be5c42c280d831fd8ad54be0d8/389-ds-base Author: Rich Megginson richm@redhat.com Date: Thu Feb 2 16:20:02 2012 -0700 Reviewed by: nhosoi (Thanks!) Branch: master Fix Description: Use slapi_sdn_get_udn(root) to get the raw DN string to pass to slapi_create_dn_string - will save a normalization if root is not normalized. Platforms tested: RHEL6 x86_64 Flag Day: no Doc impact: no

389-ds-bot commented 4 years ago

Comment from rmeggins (@richm) at 2012-02-07 21:15:07

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=788140

389-ds-bot commented 4 years ago

Comment from nkinder (@nkinder) at 2012-08-28 04:14:39

Added initial screened field value.

389-ds-bot commented 4 years ago

Comment from nkinder (@nkinder) at 2017-02-11 22:49:27

Metadata Update from @nkinder: