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.

Sign up for GitHub

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

Jump to bottom

added userPasswordAttribute to ldap config, like AD config #299

Merged
merged 1 commit into from
Jan 1, 2017
Merged

added userPasswordAttribute to ldap config, like AD config #299

merged 1 commit into from
Jan 1, 2017

Conversation

sjsmit
Copy link
Contributor

@sjsmit sjsmit commented Dec 28, 2016

added userPasswordAttribute to ldap config template, like the ad config template( see

puppet-rundeck/templates/_auth_ad.erb

Line 27 in 5e2f06e

userPasswordAttribute="<%= @auth_config['active_directory']['user_password_attribute'] %>"
) as my ldap setup required it as well.

@juniorsysadmin juniorsysadmin added the needs-feedback Further information is requested label Dec 31, 2016
@juniorsysadmin
Copy link
Member

Seems reasonable

@jyaworski jyaworski merged commit a3efa80 into voxpupuli:master Jan 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juniorsysadmin juniorsysadmin juniorsysadmin approved these changes

Assignees
No one assigned
Labels
needs-feedback Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sjsmit @juniorsysadmin @jyaworski