-
Notifications
You must be signed in to change notification settings - Fork 15
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
Support for groupOfMembers please. #13
Comments
Hello @sgparry, Thanks for your report. I'll look at that ASAP and come back to you with a test branch. Best regards, Ganael. |
Hello again, I've just pushed the 'groupOfMembers-support' branch. Could you test it and tell me if it fits your needs ? Just specify : Please test particularly those commands :
Thanks! |
I will hopefully be able to give this a try in the next few days, but the
other issue (#14) may complicate matters; I am trying to replicate an existing server set up, which does not have the groupings.
On 16 December 2021 21:32:29 GMT, Ganael Laplanche ***@***.***> wrote:
Hello again,
I've just pushed the 'groupOfMembers-support' branch. Could you test it and tell me if it fits your needs ?
Just specify :
`GCLASS="groupOfMembers"`
in your ldapscripts.conf file to get ready.
Please test particularly those commands :
- ldapaddgroup
- ldapid / ldapgid
- ldapaddusertogroup / ldapdeleteuserfromgroup
Thanks!
--
Reply to this email directly or view it on GitHub:
#13 (comment)
You are receiving this because you were mentioned.
Message ID: ***@***.***>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
scrub this comment - was looking at wrong branch... |
Ah, no pb... I was checking my patch :p |
Hello @sgparry, First of all, happy new year! Have you been able to test the proposed modifications ? |
Hello @sgparry, Any news ? |
Been super busy, but may get a chance to try it again this week. |
Hello @sgparry, Have you had time to test the patch ? |
ldapscripts currently does not support the groupOfMembers class, which unlike groupOfNames can be empty.
The text was updated successfully, but these errors were encountered: