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

Ad nested groups. #9

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

Turmio
Copy link

@Turmio Turmio commented Aug 24, 2022

I haven't check this with spaces in LDAP/AD group names. I am going to do that tomorrow. It could be an issue since I removed quotes.

@Turmio
Copy link
Author

Turmio commented Aug 25, 2022

I haven't check this with spaces in LDAP/AD group names. I am going to do that tomorrow. It could be an issue since I removed quotes.

Missing quotes were problem so those changes are reverted. Current implementation supports spaces in ldap group names.

@Turmio Turmio changed the title Ad nested groups Ad nested groups. Fixes https://github.com/BernhardLinz/zabbix-ldap-sync-bash/issues/4 Aug 25, 2022
@Turmio Turmio changed the title Ad nested groups. Fixes https://github.com/BernhardLinz/zabbix-ldap-sync-bash/issues/4 Ad nested groups. #4 Aug 25, 2022
@Turmio Turmio changed the title Ad nested groups. #4 Ad nested groups. Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant