You cannot select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
 
 
 
 
 
Go to file
Lorin Hochstein ec2fc7c983 authorized_key: Set manage_dir default value
This commit fixes a bug where the authorized_key module causes
the ~user/.ssh directory to be owned by root instead of the user,
when the manage_dir argument is not specified.

If the manage_dir argument was not specified, the module behaved as if
manage_dir was set to false, even though it's supposed to default to
true.

This module assumed that an optional argument, with no default
specified, will not be present in the module.params dictionary.

What actually seems to happen is that the argument does appear in
the module.params dictionary with a value of None.

The upside is that this line was evaluating to None instead of
true:

    manage_dir = params.get("manage_dir", True)

I fixed the problem in this particular module by explicitly specifying
the default value for the manage_dir arugment. But if this bug
occurred because of a change in behavior in AnsibleModule, then other
modules may be broken as well.
13 years ago
cloud Fixup fetch doc formatting. 13 years ago
commands Allow modules to be categorized, and also sort them when generating the documentation. 13 years ago
database Now can specify amount of time to wait_for_handoffs or ring ready instead of booleans. 13 years ago
files Fixup fetch doc formatting. 13 years ago
internal Allow modules to be categorized, and also sort them when generating the documentation. 13 years ago
inventory Allow modules to be categorized, and also sort them when generating the documentation. 13 years ago
messaging Update rabbitmq_user/vhost/parameter to pass the node to rabbitmqctl using -n flag to rabbitmqctl rather than setting the environment variable. 13 years ago
monitoring new airbrake_deployment module 13 years ago
net_infrastructure Add newline at end of file 13 years ago
network Fixup fetch doc formatting. 13 years ago
notification Merge pull request #2931 from caredotcom/flowdock_nochange 13 years ago
packaging Lots of formatting fixes 13 years ago
source_control Fix typo in git module documentation preventing it to be processed by ansible-doc 13 years ago
system authorized_key: Set manage_dir default value 13 years ago
utilities Fix empty requirements specifications in modules documentation 13 years ago
web_infrastructure Allow modules to be categorized, and also sort them when generating the documentation. 13 years ago