Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | POSIX: fixed test to only apply ACLs to non-symlinksv1.3.4 | Chris St. Pierre | 2014-02-25 | 1 | -1/+1 |
| | |||||
* | Revert "POSIX: fixed test to only apply ACLs to non-symlinks" | Chris St. Pierre | 2014-02-25 | 1 | -2/+2 |
| | | | | | | Massive typo. This reverts commit c51850b13f54d6f46e6c671e5ee1d3f0cacef727. | ||||
* | POSIX: fixed test to only apply ACLs to non-symlinks | Chris St. Pierre | 2014-02-25 | 1 | -2/+2 |
| | |||||
* | POSIX: Fix verification of symlinks | Chris St. Pierre | 2014-02-21 | 1 | -3/+3 |
| | | | | | | | | | | * Stat the link itself, not its target * Get SELinux context from the link, not the target * Don't get ACLs at all; symlinks don't have their own ACLs The first issue listed wasn't actually a bug, because none of the information queried from the target by the stat call was actually used in verification, but it's been fixed for completeness. | ||||
* | testsuite: Fixed several pylint 1.0 issues | Chris St. Pierre | 2014-02-21 | 1 | -3/+7 |
| | |||||
* | Don't strip other entries from default ACL, these are defined by mode number. | Richard Connon | 2014-02-10 | 1 | -3/+1 |
| | |||||
* | docstring for new sub-method | Richard Connon | 2014-02-04 | 1 | -0/+2 |
| | |||||
* | Except _verify_acls from pylint branches check | Richard Connon | 2014-02-04 | 1 | -1/+1 |
| | |||||
* | Removed redundant condition for empty default ACL | Richard Connon | 2014-02-04 | 1 | -8/+7 |
| | |||||
* | Fixed continue not in a loop error in previous | Richard Connon | 2014-02-04 | 1 | -1/+1 |
| | |||||
* | fix for "Too many branches" in _verify_acls | Richard Connon | 2014-02-04 | 1 | -20/+23 |
| | |||||
* | Minor changes to default ACL code | Richard Connon | 2014-02-04 | 1 | -5/+4 |
| | |||||
* | Support ACLs without a specific user/group for default owner/owning-group ↵ | Richard Connon | 2014-02-04 | 1 | -28/+52 |
| | | | | ACLs on directories | ||||
* | fixed pylint tests | Chris St. Pierre | 2014-01-30 | 1 | -2/+2 |
| | |||||
* | Augeas: ensure that entry.attrib is always a dict | Chris St. Pierre | 2014-01-29 | 1 | -1/+2 |
| | |||||
* | Client: let lxml.etree XML implementation parse very large documents | Chris St. Pierre | 2014-01-29 | 1 | -1/+9 |
| | |||||
* | added missing docstring | Chris St. Pierre | 2014-01-22 | 1 | -2/+5 |
| | |||||
* | Don't parse unicode XML with encoding | Chris St. Pierre | 2014-01-22 | 1 | -1/+10 |
| | | | | | Fix another place where a unicode XML string with an encoding declaration may be read. Cf. 0f8d403d1a86cfbfe8222662dc445e16e8f7eff9 | ||||
* | POSIX: Properly stringify ACLs with no user/group specified | Chris St. Pierre | 2014-01-16 | 1 | -1/+4 |
| | | | | | | | This is just a workaround to avoid a traceback; the real fix will involve making the POSIX tool properly handle ACLs with no user/group given, which refer to the current user/group of the file they apply to. | ||||
* | fixed indentation in previous | Richard Connon | 2014-01-05 | 1 | -1/+1 |
| | |||||
* | Fixed syntatical error in previous | Richard Connon | 2014-01-05 | 1 | -2/+2 |
| | |||||
* | Fixed group blacklist issue #150 | Richard Connon | 2014-01-05 | 1 | -0/+1 |
| | |||||
* | Augeas improvements: | Chris St. Pierre | 2013-12-11 | 1 | -10/+21 |
| | | | | | | | | | | | | | * Added ability to specify initial content for a file that doesn't exist, to avoid a messy situation where you'd have to probe for file existence and either use a Path type="file" or Path type="augeas" depending, and run Bcfg2 twice. * All commands in an Augeas path are run if *any* of them fail to verify. Previously, only commands that hadn't been run would be installed, but that had issues, particularly with the Clear command, which could pass verification but then be required during the installation phase anyway. * Miscellaneous bug fixes. | ||||
* | Augeas: avoid deprecation warning | Chris St. Pierre | 2013-12-05 | 1 | -2/+2 |
| | |||||
* | Augeas: Added docstrings, fixed some minor pylint issues | Chris St. Pierre | 2013-12-05 | 1 | -20/+91 |
| | |||||
* | Augeas: Only install unverified commands | Chris St. Pierre | 2013-12-05 | 1 | -0/+2 |
| | |||||
* | POSIX: skip loading POSIX sub-tools that raise ImportError | Chris St. Pierre | 2013-12-05 | 1 | -2/+5 |
| | | | | This mimics the behavior for "real" tools | ||||
* | Tools: new Augeas driver | Chris St. Pierre | 2013-12-05 | 1 | -0/+211 |
| | |||||
* | Remove all ACLs (including mask) from entries with no ACLs listed | Chris St. Pierre | 2013-09-12 | 1 | -0/+5 |
| | | | | | | When installing an entry with no ACLs specified, but with ACLs on the file as it exists on the filesystem, the ACL mask was preserved, even as the ACLs are deleted. | ||||
* | POSIXUsers: Fix traceback | Sol Jerome | 2013-09-06 | 1 | -1/+2 |
| | | | | Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | Fix pep8 errors | Jason Kincl | 2013-08-30 | 1 | -3/+3 |
| | |||||
* | Client: Add entries processed as important to list of entries when | Jason Kincl | 2013-08-30 | 1 | -2/+4 |
| | | | | determining if a bundle has been modified | ||||
* | SELinux: Update the matchpathcon function to use the file's mode | Jonathan Billings | 2013-08-22 | 1 | -1/+2 |
| | | | | | | | | If you don't supply a mode to the selinux.matchpathcon() function, it fails to properly look up the context in some circumstances related to context patterns in the SELinux policy. This change looks up the mode and supplies it to the function. (cherry picked from commit 20a2c9a8fb6c6ecbed259b5deccb01c01bf3304f) | ||||
* | Client: Don't modify running services if ignored | Sol Jerome | 2013-08-12 | 1 | -2/+3 |
| | | | | | | | Previously, when a bundle's contents were updated, all services listed in the bundle were stopped if the status attribute was set to 'ignore'. Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | POSIX: Fix pylint failure | Sol Jerome | 2013-08-10 | 1 | -1/+2 |
| | | | | Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | POSIX: Fix unicode file verification | Sol Jerome | 2013-08-10 | 1 | -3/+13 |
| | | | | Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | POSIXUsers: Handle unicode gecos attributes | Sol Jerome | 2013-08-02 | 1 | -1/+4 |
| | | | | Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | POSIX: Ignore permissions error on auto-created dirs | Michael Fenn | 2013-07-24 | 1 | -4/+5 |
| | | | | | | | | | | | | | | If the POSIX client tool is run as a non-root user, it is very likely that the _set_perms() call in _makedirs() will fail because it cannot set the owner of the newly-created directories. This causes _makedirs() to return False, which in turn causes POSIXFile.install() to bail out early. Applying the reasoning in <https://github.com/Bcfg2/bcfg2/pull/108> the freebie directories created by _makedirs should have mode and ownership done on a best-effort basis. If a user needs parent directories to have a specific ownership and mode, then they should specify that directory in their configuration. | ||||
* | Merge branch 'fix-bootstatus' of https://github.com/AlexanderS/bcfg2 into maint | Sol Jerome | 2013-07-17 | 1 | -1/+1 |
|\ | |||||
| * | Bcfg2/Client/Tools: do not get bootstatus directly from the entry | Alexander Sulfrian | 2013-07-17 | 2 | -2/+2 |
| | | | | | | | | | | Bootstatus is optional and should have the value of status if not specified. This is handled by get_bootstatus. | ||||
* | | POSIXUsers: remove usermod -m option, which is too new | Chris St. Pierre | 2013-07-16 | 1 | -1/+0 |
| | | |||||
* | | Merge branch 'utf8-fixes' of https://github.com/stgraber/bcfg2 into maint | Sol Jerome | 2013-07-12 | 1 | -0/+3 |
|\ \ | |||||
| * | | Don't crash when a file diff contains utf-8 chars | Stéphane Graber | 2013-07-12 | 1 | -0/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a crash when the client tries to render the diff of a file which includes unicode characters. This change assumes that utf-8 is the default encoding, which looking at the rest of the code appears to be a safe bet. Signed-off-by: Stéphane Graber <stgraber@ubuntu.com> | ||||
* | | | Merge pull request #114 from igorwidlinski/patch-1 | Chris St. Pierre | 2013-07-12 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | Chkconfig should not set service to "on" on every run level | ||||
| * | | Stop adding chkconfig service to every run level | igorwidlinski | 2013-07-04 | 1 | -1/+1 |
| | | | | | | | | | Let chkconfig assign default runlevels. | ||||
* | | | Copy xml elements before appending to tree. This prevents modified entries ↵ | Tim Laszlo | 2013-07-10 | 1 | -1/+2 |
| | | | | | | | | | | | | from being added | ||||
* | | | RcUpdate: Use get_bootstatus | Sol Jerome | 2013-07-08 | 1 | -1/+1 |
|/ / | | | | | | | Signed-off-by: Sol Jerome <sol.jerome@gmail.com> | ||||
* | | Chkconfig: give --list option to chkconfig rather than sending it on stdin | Michael Fenn | 2013-07-02 | 1 | -2/+2 |
| | | |||||
* | | New approach, just create nwe directories with mode 0755 regardless | Michael Fenn | 2013-07-01 | 1 | -9/+3 |
|/ | |||||
* | Client/Frame: avoid use of lxml-only getparent() | Alexander Sulfrian | 2013-06-29 | 1 | -1/+4 |
| | | | | | | getparent is not supported by xml.etree so try to emulate it with a generated parent_map. This is the only possibility, because xml.etree does not store references to the parent elements. |