valitydev/salt
14
0
Fork 0
mirror of https://github.com/valitydev/salt.git synced 2024-11-08 01:18:58 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

cp.get_url: fix dest=None behaviour with salt:// URL

Browse Source
This commit is contained in:
Denys Havrysh 2016-10-10 17:56:37 +03:00
parent 3ce4897b97
commit 99cf3038cc
2 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
salt/fileclient.py
View File

@ -556,8 +556,12 @@ class Client(object):
return url_data.path
if url_data.scheme == 'salt':
return self.get_file(
url, dest, makedirs, saltenv, cachedir=cachedir)
cached = self.get_file(url, dest, makedirs, saltenv, cachedir=cachedir)
if dest is None:
with salt.utils.fopen(cached, 'r') as fp_:
data = fp_.read()
return data
return cached
if dest:
destdir = os.path.dirname(dest)
if not os.path.isdir(destdir):

10
salt/modules/cp.py
View File

@ -309,11 +309,11 @@ def get_dir(path, dest, saltenv='base', template=None, gzip=None, env=None, **kw
def get_url(path, dest, saltenv='base', env=None):
'''
Used to get a single file from a URL.
Used to get a single file from a URL
The default behaviuor is to write the fetched file to the given
destination path. To simply return the text contents instead, set destination to
None.
The default behaviour is to write the fetched file to the given
destination path. To simply return the file contents instead, set
destination to ``None``.
CLI Example:
@ -331,7 +331,7 @@ def get_url(path, dest, saltenv='base', env=None):
# Backwards compatibility
saltenv = env
if dest:
if isinstance(dest, str):
return _client().get_url(path, dest, False, saltenv)
else:
return _client().get_url(path, None, False, saltenv, no_cache=True)