ansible

Commit Graph

Author	SHA1	Message	Date
Rick Elrod	6e3271aa61	[stable-2.10] Revert default mode changes (#71260 ) * Revert "[stable-2.10] Revert "Change default file permissions so they are not world readable (#70221) (#70824)" (#71236)" This reverts commit `c968020d52`. * Revert "Remove porting guide entry related to reverted change (#71242)" This reverts commit `006a21eae2`.	5 years ago
Sam Doran	c968020d52	[stable-2.10] Revert "Change default file permissions so they are not world readable (#70221 ) (#70824 )" (#71236 ) * [stable-2.10] Revert "Fix warning for new default permissions when mode is not specified (#70976) (#70985)" This reverts commit `5cb96087e6`. * [stable-2.10] Revert "Change default file permissions so they are not world readable (#70221) (#70824)" This reverts commit `7e4cffc5d2`.	5 years ago
Sam Doran	7e4cffc5d2	[stable-2.10] Change default file permissions so they are not world readable (#70221 ) (#70824 ) * Change default file permissions so they are not world readable CVE-2020-1736 Set the default permissions for files we create with atomic_move() to 0o0660. Track which files we create that did not exist and warn if the module supports 'mode' and it was not specified and the module did not call set_mode_if_different(). This allows the user to take action and specify a mode rather than using the defaults. A code audit is needed to find all instances of modules that call atomic_move() but do not call set_mode_if_different(). The findings need to be documented in a changelog since we are not warning. Warning in those instances would be frustrating to the user since they have no way to change the module code. - use a set for storing list of created files - just check the argument spac and params rather than using another property - improve the warning message to include the default permissions. (cherry picked from commit `5260527c4a`) Co-authored-by: Sam Doran <sdoran@redhat.com>	5 years ago
Brian Coca	ac509d489b	Revert "stricter permissions on atomic_move when creating new file (#68970 )" (#68983 ) This reverts commit `566f2467f6`.	6 years ago
Brian Coca	566f2467f6	stricter permissions on atomic_move when creating new file (#68970 ) fixes #67794 updated some tests that expected previous defaults CVE-2020-1736	6 years ago
Matt Martz	697b566971	Update units to pass on macOS (#60435 ) * Update units to pass on macOS. Fixes #27810 * raising=False	6 years ago
Sam Doran	2a98faee2b	Move _handle_aliases() out of basic.py (#48578 ) Refinements: - return legal_inputs and update class properties - remove redundant arguments from method and handle in caller - add better exception types to method * Add unit tests for handle_aliases	7 years ago
Sviatoslav Sydorenko	f1e67c3328	Bugfix/ fix mocker patch in tests (#52372 ) * 🐛 Fix invalid os.stat mock in tests * 🐛 Fix leaking mock patch in tests Closes #52347	7 years ago
Toshio Kuratomi	370a7ace4b	Split basic units (#33510 ) Split the one monolithic test for basic.py into several files * Split test_basic.py along categories. This is preliminary to get a handle on things. Eventually we may want to further split it so each file is only testing a single function. * Cleanup unused imports from splitting test_basic.py * Port atomic_move test to pytest. Working on getting rid of need to maintain procenv * Split a test of symbolic_mode_to_octal to follow unittest best practices Each test should only invoke the function under test once * Port test_argument_spec to pytest. * Fix suboptions failure	8 years ago

9 Commits (6e3271aa61bae4b9e4b51801cfe2723bdd7298cc)