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

fix various issues with the authorized_keys module #522

Closed
bradobro opened this issue Jul 2, 2012 · 3 comments
Closed

fix various issues with the authorized_keys module #522

bradobro opened this issue Jul 2, 2012 · 3 comments

Comments

@bradobro
Copy link
Contributor

bradobro commented Jul 2, 2012

Fix bug where key state=absent will write a non-existent file. No need to change resource in that case, so be idempotent.
@mpdehaan
Copy link
Contributor

mpdehaan commented Jul 2, 2012

Merging tickets:

Remove comment ("#Authorized Keys File created by Ansible.") this module currently places in the authorized_key module.

@mpdehaan
Copy link
Contributor

mpdehaan commented Jul 2, 2012

also:

the authorized_keys module should work under Python 2.4 and earlier -- remove the with blocks and test

@mpdehaan
Copy link
Contributor

mpdehaan commented Jul 2, 2012

closing per PR

@mpdehaan mpdehaan closed this as completed Jul 2, 2012
mpdehaan pushed a commit to mpdehaan/ansible that referenced this issue Jul 17, 2012
@bradobro
Fixes ansible#522, no longer creates comment in ~/.ssh/authorized_key…
be9ff7f 
…s, does not create directory or file if state==absent.
mpdehaan pushed a commit to ansible/ansible-modules-core that referenced this issue Sep 26, 2014
@bradobro
Fixes ansible/ansible#522, no longer creates comment in ~/.ssh/author…
9e13c1e 
…ized_keys, does not create directory or file if state==absent.
@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.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mpdehaan @bradobro