From 33773624fbcbdc529db1965c68c8b2f0dea86b20 Mon Sep 17 00:00:00 2001 From: Jordan Borean Date: Wed, 7 Mar 2018 08:55:39 +1000 Subject: [PATCH] win_wait_for: fix local port check on port not accessible externally (#37093) * win_wait_for: use loopback IP instead of hostname if 127.0.0.1 is used (#36762) * win_wait_for: use loopback IP instead of hostname if 127.0.0.1 is used * removed reverse dns lookup in port check (cherry picked from commit 67ffde4ac1be60cc851709df6a4ea2f31861610e) * Added changelog for win_wait_for local port fix --- changelogs/fragments/win_wait_for-local-port-fix.yaml | 2 ++ lib/ansible/modules/windows/win_wait_for.ps1 | 10 +--------- 2 files changed, 3 insertions(+), 9 deletions(-) create mode 100644 changelogs/fragments/win_wait_for-local-port-fix.yaml diff --git a/changelogs/fragments/win_wait_for-local-port-fix.yaml b/changelogs/fragments/win_wait_for-local-port-fix.yaml new file mode 100644 index 00000000000..d70b6ec6d58 --- /dev/null +++ b/changelogs/fragments/win_wait_for-local-port-fix.yaml @@ -0,0 +1,2 @@ +bugfixes: +- win_wait_for - fixed issue when trying to check a localport when the port is not available externally diff --git a/lib/ansible/modules/windows/win_wait_for.ps1 b/lib/ansible/modules/windows/win_wait_for.ps1 index dee606ff33e..66fd884deaf 100644 --- a/lib/ansible/modules/windows/win_wait_for.ps1 +++ b/lib/ansible/modules/windows/win_wait_for.ps1 @@ -54,17 +54,9 @@ if ($port -ne $null) { } Function Test-Port($hostname, $port) { - # try and resolve the IP/Host, if it fails then just use the host passed in - try { - $resolve_hostname = ([System.Net.Dns]::GetHostEntry($hostname)).HostName - } catch { - # oh well just use the IP addres - $resolve_hostname = $hostname - } - $timeout = $connect_timeout * 1000 $socket = New-Object -TypeName System.Net.Sockets.TcpClient - $connect = $socket.BeginConnect($resolve_hostname, $port, $null, $null) + $connect = $socket.BeginConnect($hostname, $port, $null, $null) $wait = $connect.AsyncWaitHandle.WaitOne($timeout, $false) if ($wait) {