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: use default user_map when looking of host groups for GPO #7591

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

Conversation

sumit-bose
Copy link
Contributor

Use the default AD user attribute map to lookup the group membership of the
AD host object. This should help to avoid issues if user attributes are
overwritten in the user attribute map.

Resolves: #7590

Copy link
Contributor

@thalman thalman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

To allow to lookup group memberships of other objects similar to user
objects but with different attribute mappings, e.g. host objects in AD,
a new option to provide an alternative attribute map is added.

Resolves: SSSD#7590
Use the default AD user attribute map to lookup the group membership of
the AD host object. This should help to avoid issues if user attributes
are overwritten in the user attribute map.

Resolves: SSSD#7590
Copy link
Contributor

@justin-stephenson justin-stephenson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted backport-to-stable Targets also latest stable branch Bugzilla
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GPO access control might fail if ldap_user_name is set
4 participants