diff options
author | James E. Blair <jim@acmegating.com> | 2022-04-06 16:05:44 -0700 |
---|---|---|
committer | James E. Blair <jim@acmegating.com> | 2022-04-13 14:17:04 -0700 |
commit | e533d646829c05b5d3c00f1e0683fb7fd1a7d5cd (patch) | |
tree | 2367abe28a9d3a4d032f4bc2aa51ecd95bad5464 /zuul/ansible | |
parent | 810c9e31ef4573a3b1769e53bf04c623c8462b42 (diff) | |
download | zuul-e533d646829c05b5d3c00f1e0683fb7fd1a7d5cd.tar.gz |
Remove workspace path check from zuul_return
This restriction is no longer necessary since it was used to check
that we weren't writing something outside of the work/ dir, but that
is now permitted.
Change-Id: Ic447762c7654cf9ac8e9f8bcecc51e8b76b8e67f
Diffstat (limited to 'zuul/ansible')
-rw-r--r-- | zuul/ansible/base/actiongeneral/zuul_return.py | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/zuul/ansible/base/actiongeneral/zuul_return.py b/zuul/ansible/base/actiongeneral/zuul_return.py index 2114adba4..ba6c5cc31 100644 --- a/zuul/ansible/base/actiongeneral/zuul_return.py +++ b/zuul/ansible/base/actiongeneral/zuul_return.py @@ -20,8 +20,6 @@ from copy import deepcopy from ansible.plugins.action import ActionBase -from zuul.ansible import paths - def merge_dict(dict_a, dict_b): """ @@ -167,9 +165,6 @@ class ActionModule(ActionBase): path = os.path.join(os.environ['ZUUL_JOBDIR'], 'work', 'results.json') - if not paths._is_safe_path(path, allow_trusted=False): - return paths._fail_dict(path) - set_value( path, self._task.args.get('data'), |