#326 memberOf attribute and plugin behaviour between sub-suffixes
Closed: Fixed None Opened 7 years ago by nkinder.

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]:

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.

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

Added initial screened field value.

Metadata Update from @mreynolds:
- Issue assigned to mreynolds
- Issue set to the milestone: 1.2.11

2 years ago

Login to comment on this ticket.

Metadata