-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[bitnami/openldap] Enable overlays #50524
Comments
so I have successfully turned a few configlines into around 60 ldif lines:
However, placing this in /ldifs does not seem to have the desired effect. And honestly, this seems to be the wrong approach. i.e. to take two lines of very easy to understand config and turn them into some archaic block of 60 lines of ldif. I seem to be handing the next person who looks at these configs a major headache. |
perhaps it requires to be ran as schema? (EDIT: the
does not seem to be working either. |
Minimally I have tried having just two files in /shemas
and
at this point I can cause the openldap instance to crash by requesting a memberof attribute:
^returns fine, But this one crashes the server:
the logs have nothing useful to say about why it died:
my docker-compose.yml at this point was:
|
I have made an example repo for demonstration purposes: |
Hi @joshuacox , I recently had a similar requirement to use an overlay, Sync Provider, not enabled OOTB. Happy to create a PR for this if that approach would work? |
@jonnoss1 I'd be very happy to test a PR! tyvm |
Hi @joshuacox, I've pushed the changes here.
|
@jonnoss1 awesome on actually implementing the environment variables. However, I do seem to be getting the same crashing situation when I implement:
which I got directly from the man page
but uid alone returns just fine:
|
Hi @joshuacox , What schema are you using to add
|
I believe I'm getting it by using dyngroup: Here is my full docker-compose.yml:
|
Looks like you also need the memberOf overlay . Pushed another change earlier today to add support for this.
Ldif for dynamic group
I get the following search results:
Not really familiar with the dynlist overlay so can't even guess what the issue is with this ATTRSET |
Wow, this is fantastic! @jonnoss1 you have done it! Please submit a PR and get this merged in! EDIT: just adding in my final docker-compose.yml
|
@jonnoss1 just one slight note, in trying to replicate what you have done here before your PR gets merged, I run into this error:
where my schemas directory looks like this:
Removing the
But I am wondering what schema is supplying that |
Hi @joshuacox, Maybe a couple questions before we forge on with a PR.
This one is definitely incorrect.
Should be something like this
This is likely causing some kind of problem. Re: where the schema's come from:
Are you only looking be be able to run a search like this on the memberOf attribute: Or do you additionally require the ability to define dynamic lists using this pattern:
|
I believe that is generated when using this env var:
which was the original example you gave, that is where I got confused. But my current problem is still crashing when I execute something like:
in the logs I can see the 'crash' at exit 0, though I'm not certain why openldap thought it appropriate to throw a zero there, as it seems to me to be worthy of an exit 1 or higher.
As to whether I need the dynamic group, I don't foresee it immediately being necessary, but I also didn't realize I needed a few things here before getting further. My end goal is to be able to use a user filter in gitea something like:
and an admin filter of:
EDIT: I did make the changes to the olcDynListAttrSet here and I added a crash.sh to document that command. |
Hi @joshuacox, I've created a PR to add support for Reverse Group Membership Maintenance aka Based on the example searches you are trying to perform this appears to be the cleanest way to add support for this specific reciprocal attribute. Can always look at J |
Thank you for submitting the associated Pull Request. Our team will review and provide feedback. Once the PR is merged, the issue will automatically close. Your contribution is greatly appreciated! |
The problem with memberOf is that it is deprecated, will be removed in the future, and it is discrouraged on replicated setups. from man slapo-memberof:
After a week of not looking at this issue (because the team I was working with decided to go with freeIPA instead), I came back and looked at the comment here I realized I needed to alter this line:
to this line:
so the example repo here is fixed. And all memberOf functionality is supplied by dynlist now:
|
Hi, The Openldap Team changes her mind about memberof overlay : it is no more deprecated (this announce) So this PR is very interesting : can you merge it ? Thank you |
Name and Version
bitname/openldap:2.6.6
What is the problem this feature will solve?
The documentation points to adding dynlist like so:
to slapd.conf However there is no slapd.conf in the bitnami container. as we are using /bitnami/openldap/slapd.d
The documentation points to doing something like this to convert:
But again I don't have a slapd.conf file to convert, or is there a base file I can use for this purpose? or is there a way to merge in an overlay?
What is the feature you are proposing to solve the problem?
Just an explanation of how to add in an overlay would be fantastic. Potentially adding it to the documentaiton. Or maybe even an environment variable:
I would imagine most would need configuration lines as well, like:
or something similar.
What alternatives have you considered?
just mapping in a directory and trying to slaptest a config that I might be able to ldifs out of.
with some tweaking I might be able to hack together a solution.
minimal slapd.conf (EDIT: now it works, but I am uncertain what all I need to extract from it)
so now I can get a slapcat:
It certainly is a lot of data, should I just be diffing that and slapcat on a slapd.conf of:
The text was updated successfully, but these errors were encountered: