Commit Graph

478 Commits (570e74bb1a5ce7021b44ed37bfb10934932b6d80)

Author SHA1 Message Date
Brian Coca f12a2135ca Merge pull request #1525 from mjuenema/devel
Removed check whether destination directory is writable from files/un…
9 years ago
James Cammarata 310cf77edc Merge pull request #1947 from astorije/astorije/fix-acl
Fix the acl module
9 years ago
Brian Coca 68ab025dac minor doc fixes 9 years ago
Jérémie Astori 72fb7a0a17 Fix physical walk on acl module for Linux
`-h` is for BSD [1] while `-P`/`--physical` is for Linux [2].
This commit fixes that option now that acl module is (temporarily)
only supported for Linux.
I will re-add `-h` when fixing BSD support.

[1]: http://www.freebsd.org/cgi/man.cgi?format=html&query=setfacl(1)
[2]: http://linuxcommand.org/man_pages/setfacl1.html
9 years ago
Jérémie Astori 8eefd44aef Make sure entry is not sent when acl state=query 9 years ago
Jérémie Astori 0e659ad872 Make sure permission-less entries are accepted when state=absent
Also, remove that try condition as, at that stage, no permissions with
other than 2 or 3 fields are sent to the function.
9 years ago
Jérémie Astori e95bcaeb8a Remove support for `d[efault]:` in entry permissions
It is not documented in [the Ansible doc page][1] nor
[the BSD setfacl man entry][2] (which means it might not be compatible
with BSD) so removing it does not break the API.

On the other hand, it does not conform with POSIX 1003.1e DRAFT
STANDARD 17 according to the [Linux setfacl man entry][3] so safer to
remove.

Finally, the most important reason: in non POSIX 1003.e mode, only ACL
entries without the permissions field are accepted, so having an
optional field here is very much error-prone.

[1]: http://docs.ansible.com/ansible/acl_module.html
[2]: http://www.freebsd.org/cgi/man.cgi?format=html&query=setfacl(1)
[3]: http://linuxcommand.org/man_pages/setfacl1.html
9 years ago
Brian Coca f925f7e01f Merge pull request #1734 from freym/assemble-file-validation
assemble: add file validation support.
9 years ago
Jérémie Astori 3ac990556d Fix wrong expectation regarding entry format in acl module 9 years ago
Jérémie Astori 421d3f12cf Fix wrong processing of lines returned by setfacl in test mode 9 years ago
Jérémie Astori 50c6425673 Fix minor whitespace issues 9 years ago
Brian Coca d61fde42ef updated encoding comment and expanded authors field 9 years ago
Brian Coca d4a39d23df Merge pull request #150 from astorije/recursive_acl
Add recursive support for ACL module
9 years ago
Jérémie Astori acfc2166b0 Set version number in recursive option description 9 years ago
Jérémie Astori f8027a7f0a Ensure Python 2.4 compatibility and Linux-restricted support
- Make build_entry compatible with Python 2.4
- Re-add missing warning/comment that was forgotten while refactoring
- Replace `all()` with a good ol' for-loop Python 2.4 compatibility
- Make a condition check more explicit (when `state` is `query`)
- Make sure this module can only be run with on a Linux distribution
- Add a note about Linux-only support in the documentation
- Set the version in which recursive support was added, 2.0
9 years ago
Marius Gedminas 9bcf69a155 Fix obvious copy-paste error in module docs 9 years ago
Toshio Kuratomi f39e5ffa1d Modify the default for synchronize's dest_port again 9 years ago
Toshio Kuratomi ed2d8121b3 Update docs to reflect precedence for synchronize's dest_port param 9 years ago
Toshio Kuratomi 0071fd0bf7 Allow mode as a synchronize parameter (handled in action plugin) 9 years ago
Brian Coca 697307f88c Merge pull request #1670 from mikeputnam/issue-1643-gpl-header
Add missing GPLv3 License header
9 years ago
Toshio Kuratomi 19741d75eb Fi the local_rsync_path parameter used by the action plugin 9 years ago
James Cammarata 2bfddb015d In copy, set the mode before running th validation
Because some programs that do validation (like visudo) may require
the permissions to be more restricted.

Fixes ansible/ansible#11385
9 years ago
Brian Coca 9b8470ae95 minor doc fixes 9 years ago
Brian Coca c688408385 fixed fragment docs 9 years ago
Jeff Widman fd6518179b Move validate command into doc fragment and fix a few typos 9 years ago
Brian Coca 46fa43d9e0 Merge pull request #988 from isaacsimmons/ini-file-catch-interpolation-error
Handle ini file properties that contain interpolation errors
9 years ago
Patrik Lundin 3849a6d87b synchronize: add flag for verifying target host.
Add the possibility to verify the target host using a "verify_host"
flag. It is disabled by default to not change the module behaviour.
9 years ago
Brian Coca 2e77041dae Merge pull request #1654 from bcoca/file_dir_exceptions
now captures any exceptions when trying to create directories
9 years ago
Matthias Frey 68a3c6ece4 assemble: add file validation support. 9 years ago
Brian Coca e916b04e91 Also document in example that unarchive download was added in 2.0 9 years ago
Mitchell Ludwig 9e11f5fd15 Improved stat documentation 9 years ago
Brian Coca e33f093075 added ignore hidden to assemble 9 years ago
Jérémie Astori ae1126c217 Add recursive support for ACL module (fix #5053, fix #5550, fix #7276) 10 years ago
Mike Putnam 6c0e264806 Add missing GPLv3 License header
Fixes #1643
10 years ago
Brian Coca 37506141a3 Merge pull request #1613 from verm666/issue-1575
unarchive: fix work with 0 bytes archives
10 years ago
Brian Coca ed179fe379 now captures any exceptions when trying to create directories 10 years ago
Matthew Gilliard 910728f6c3 Handle race condition in directory creation.
If we try to make a directory, but someone else creates the directory
at the same time as us, we don't need to raise that error to the user.
They asked for the directory to exist, and now it does. This fixes
the race condition which was causing that error to be raised, and
closes #1648.
10 years ago
verm666 8deee99fcc unarchive: fix @bcoca's remarks, issue #1575 10 years ago
Brian Coca 433ca93c42 Merge pull request #1605 from vladimirmartsul/patch-1
Add "force" description
10 years ago
Brian Coca 13116d8ac0 Merge pull request #839 from gimoh/patch-1
Do not insert extra newline if line already contains it
10 years ago
verm666 725a7b2f59 unarchive: fix work with 0 bytes archives
This change is in response to issue #1575
10 years ago
Toshio Kuratomi a5bba2488f Update version_added to 2.0 for the partial option 10 years ago
Vladimir Martsul 2206477b73 Add "force" description
Add "force" option description
10 years ago
Juan Picca 892212b9c4 synchronize module: add partial option 10 years ago
Toshio Kuratomi c5324f54e6 Bump amount of file to download in a chunk to 64k. 10 years ago
Toshio Kuratomi daa1472ccf Merge pull request #1448 from Jmainguy/unarchive_http
add download ability to unarchive module
10 years ago
Jonathan Mainguy 58ef71fc84 add download ability to unarchive module 10 years ago
Ritesh Khadgaray 81528e6834 Add the ability to grep for content 10 years ago
Markus Juenemann 0db3206cae Removed check whether destination directory is writable from files/unarchive.py.
This check will prevent extraction of an archive if the archive does not
actually write to the destination directory but only writes to any writable
sub-directories of it. The underlying tar command will report errors
should it try to write to read-only directories.
10 years ago
Greg DeKoenigsberg 2a5f0bde87 Proper author info for all remaining modules 10 years ago