Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for memberOf overlay #52239

Closed
wants to merge 1 commit into from
Closed

Conversation

jonnoss1
Copy link
Contributor

Description of the change

This change extends the existing [bitnami/openldap:2.6] container deployment with configurable support for the existing memberOf module:

  • Reverse Group Membership Maintenance memberOf

Benefits

Future users of the [bitnami/openldap] container will be able to access the Reverse Group Membership Maintenance module and functionality with minimal configuration.

Possible drawbacks

Applicable issues

  • fixes #

Additional information

This change enabled the capability required to support the use case described in this issue

Signed-off-by: Jonno Sanderson-Smith <[email protected]>
@github-actions github-actions bot added the triage Triage is needed label Oct 30, 2023
@bitnami-bot bitnami-bot requested a review from carrodher October 30, 2023 15:35
@carrodher carrodher added the verify Execute verification workflow for these changes label Oct 30, 2023
@github-actions github-actions bot added in-progress and removed triage Triage is needed labels Oct 30, 2023
@bitnami-bot bitnami-bot removed the request for review from carrodher October 30, 2023 17:11
@bitnami-bot bitnami-bot requested a review from mdhont October 30, 2023 17:11
@mdhont
Copy link
Contributor

mdhont commented Nov 2, 2023

As mentioned in #50524 (comment) it seems memberOf will be deprecated in future releases.

@jonnoss1
Copy link
Contributor Author

jonnoss1 commented Nov 2, 2023

All good. I'll look at the changes for the dynlist overlay again when I have some time to kill.

@jonnoss1 jonnoss1 closed this Nov 2, 2023
@CrnTeam
Copy link

CrnTeam commented Dec 12, 2023

@carrodher I am interested by this PR, what is missing to merge this new feature ?

@carrodher
Copy link
Member

According to the latest comment, from @mdhont it seems memberOf will be deprecated in future releases.

@jonnoss1
Copy link
Contributor Author

The memberOf overlay was deprecated from 2.5, and based on the 2.5 & 2.6 man pages, support will be dropped in future releases.
I've got changes for the dynlist overlay stashed; will create another PR with them and link here as soon as I get a gap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
openldap solved verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants