From 6e035a3e94efc18d7547b5b12d0cf520a8408106 Mon Sep 17 00:00:00 2001 From: Petr Mifek Date: Sat, 10 Oct 2015 00:01:22 +0200 Subject: [PATCH] Move nested function test for ipv6 to top level in synchronize module. --- lib/ansible/plugins/action/synchronize.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/ansible/plugins/action/synchronize.py b/lib/ansible/plugins/action/synchronize.py index eb7848dad05..c80523c0c00 100644 --- a/lib/ansible/plugins/action/synchronize.py +++ b/lib/ansible/plugins/action/synchronize.py @@ -44,17 +44,17 @@ class ActionModule(ActionBase): return path + def _host_is_ipv6_address(self, host): + return ':' in host + def _format_rsync_rsh_target(self, host, path, user): ''' formats rsync rsh target, escaping ipv6 addresses if needed ''' - def _needs_ipv6_brackets(host): - return ':' in host - user_prefix = '' if user: user_prefix = '%s@' % (user, ) - if _needs_ipv6_brackets(host): + if self._host_is_ipv6_address(host): return '[%s%s]:%s' % (user_prefix, host, path) else: return '%s%s:%s' % (user_prefix, host, path)