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

Add support for crypted passwords to password lookup #2939

Merged
merged 2 commits into from May 18, 2013

Conversation

@mmoya
Copy link
Contributor

@mmoya mmoya commented May 16, 2013

Added new parameter 'encrypt' with same semantics from that of
vars_prompt. When encryption is requested a random salt will be
generated and stored along the password in the form:
' salt='.

Also store password with an ending '\n' for easier looking at files
with console tools. File content was being already rstripped so this
is harmless.

Includes a fix for Javier's name following his request.

Added new parameter 'encrypt' with same semantics from that of
vars_prompt. When encryption is requested a random salt will be
generated and stored along the password in the form:
'<password> salt=<salt>'.

Also store passwords with an ending '\n' for easier looking at files
with console tools. File content was being already rstripped so this
is harmless.
@candeira

This comment has been minimized.

Si me corriges el nombre (s/Javie/Javier/), te lo agradezco.

mpdehaan added a commit that referenced this pull request May 18, 2013
Add support for crypted passwords to password lookup
@mpdehaan mpdehaan merged commit 8475a92 into ansible:devel May 18, 2013
@mpdehaan
Copy link
Contributor

@mpdehaan mpdehaan commented May 18, 2013

Outstanding, glad to see this one!

@mmoya mmoya deleted the mmoya:lookup-crypted-passwds branch May 18, 2013
@mmoya mmoya mentioned this pull request Jun 6, 2013
robinro pushed a commit to robinro/ansible that referenced this pull request Dec 9, 2016
@ansible ansible locked and limited conversation to collaborators Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants