Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

optimize ldap sync at config generation (Fix #6949) #7130

Closed
wants to merge 2 commits into from
Closed

optimize ldap sync at config generation (Fix #6949) #7130

wants to merge 2 commits into from

Conversation

garnier-quentin
Copy link
Contributor

@garnier-quentin garnier-quentin commented Jan 16, 2019 •
edited
Loading

Some rules about Centreon and LDAP:

  • Users need to be connected at last one time to centreon UI. Otherwise, the user don't get notification from centreon-engine. (we could add import users directly from a LDAP contact group linked in centreon)
  • Centreon doesn't manage contact group name changed. We cannot know if a "group X" in "OU=X" is the same than "group X" in "OU=Y".
  • Centreon manage if an user added/moved in LDAP groups configured in Centreon.

In the global ldap sync, we should manage:

  • users is disabled in LDAP (need to map a ldap attribute in centreon config), it should be disabled in centreon

sc979
sc979 suggested changes Jan 23, 2019
View reviewed changes
www/class/centreonContactgroup.class.php Outdated Show resolved Hide resolved
@lpinsivy lpinsivy mentioned this pull request Feb 18, 2019
Closed
@miteto miteto self-requested a review March 14, 2019 09:04
@garnier-quentin
Copy link
Contributor Author

Already merged in that PR: https://github.com/centreon/centreon/pull/7278
So it's ok :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sc979 sc979 sc979 approved these changes

@victorvassilev victorvassilev victorvassilev approved these changes

@miteto miteto miteto approved these changes

Assignees
No one assigned
Labels
area/ldap kind/enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@garnier-quentin @miteto @victorvassilev @sc979 @lpinsivy