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

Enable "roleUsernameMemberAttribute" for LDAP auth (issue #62) #65

Conversation

wcooley
Copy link
Contributor

@wcooley wcooley commented May 12, 2015

For sites using RFC 2307 posixGroup for roles, the roleMemberAttribute does not work, because the posixGroup memberUid attributes are usernames and not DNs; the roleUsernameMemberAttribute is for this.

wcooley and others added 2 commits May 11, 2015 17:22
For sites using RFC 2307 posixGroup for roles, the *roleMemberAttribute*
does not work, because the posixGroup *memberUid* attributes are
usernames and not DNs; the *roleUsernameMemberAttribute* is for this.

Since *role_member_attribute* is already specified in params, it is
retained as the default unless *role_username_member_attribute* is set.
I think these two are mutually exclusive, so there should be no case
where they would both be enabled.
liamjbennett pushed a commit that referenced this pull request May 15, 2015
…e_member_attribute

Enable "roleUsernameMemberAttribute" for LDAP auth (issue #62)
@liamjbennett liamjbennett merged commit 5bd2537 into voxpupuli:master May 15, 2015
@wcooley wcooley deleted the wcooley/feature-ldap-role_username_member_attribute branch May 28, 2015 19:19
bastelfreak added a commit to bastelfreak/puppet-rundeck that referenced this pull request Jan 13, 2017
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.

2 participants