https://bugzilla.redhat.com/show_bug.cgi?id=804153 (Fedora)
Description of problem: It seems that the memberOf attribute handling by the memberOf plugin limited to objects inside the same subsuffix. If my group suffix is based on a different database from my user suffix, then the memberof plugin does not update the memberof attribute. The plugin works only if users and groups are locatred in the same suffix. Version-Release number of selected component (if applicable): v. 1.2.10.4 How reproducible: see description Steps to Reproduce: 1. 2. 3. Actual results: Unable to use this plugin because memberof does not work properly. Expected results: Add the correct attribute to the user entry Additional info:
Created a new config setting in the plugin to turn this on and off: "memberOfAllBackends: on". Sending out for review.
Why do we need to include slap.h now? We should try to avoid using anything other than public header files in plug-ins.
I included slap.h for "pblock_done". I was trying to avoid changes to the core code, but I can just move it to slapi-plugin.h. Or create a slapi_pblock_done() to be consistent with the slapi api?
Replying to [comment:7 mreynolds]:
Does slapi_pblock_init() do what you need? It calls pblock_done() internally, then re-initliazes the pblock for reuse.
revised to use slapi_pblock_init 0001-Ticket-326-MemberOf-plugin-should-work-on-all-backen.patch
Replying to [comment:8 nkinder]:
Replying to [comment:7 mreynolds]: I included slap.h for "pblock_done". I was trying to avoid changes to the core code, but I can just move it to slapi-plugin.h. Or create a slapi_pblock_done() to be consistent with the slapi api? Does slapi_pblock_init() do what you need? It calls pblock_done() internally, then re-initliazes the pblock for reuse.
Yes it does, thanks.
I guess Nathan should have changed the Review pull down to nack. ;)
[mareynol@localhost memberof]$ git merge ticket326 Updating a5ed824..118e483 Fast-forward ldap/servers/plugins/memberof/memberof.c | 233 ++++++++++++----------- ldap/servers/plugins/memberof/memberof.h | 3 + ldap/servers/plugins/memberof/memberof_config.c | 33 +++- 3 files changed, 160 insertions(+), 109 deletions(-)
[mareynol@localhost memberof]$ git push origin master Counting objects: 17, done. Delta compression using up to 4 threads. Compressing objects: 100% (9/9), done. Writing objects: 100% (9/9), 2.24 KiB, done. Total 9 (delta 6), reused 0 (delta 0) To ssh://git.fedorahosted.org/git/389/ds.git a5ed824..118e483 master -> master
Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=833222
Added initial screened field value.
Metadata Update from @mreynolds: - Issue assigned to mreynolds - Issue set to the milestone: 1.2.11
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/326
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.