From f2c8b4b9009541b5a19d90b2ed3bc106f46dcdb4 Mon Sep 17 00:00:00 2001 From: Jamil Date: Tue, 26 Jul 2022 21:23:08 -0700 Subject: [PATCH] check ifconfig.me return before use (#855) --- omnibus/cookbooks/firezone/recipes/network.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/omnibus/cookbooks/firezone/recipes/network.rb b/omnibus/cookbooks/firezone/recipes/network.rb index 38ac84bc5..0d0817d48 100644 --- a/omnibus/cookbooks/firezone/recipes/network.rb +++ b/omnibus/cookbooks/firezone/recipes/network.rb @@ -21,8 +21,10 @@ unless node['firezone']['wireguard']['endpoint'] # Figure out a sane default endpoint IP address egress_ip = begin - Net::HTTP.get('ifconfig.me', '/') - rescue StandardError + # Parse to ensure we get back something usable. + URI.parse(Net::HTTP.get('ifconfig.me', '/')) + rescue StandardError => e + Chef::Log.warn(e) nil end node.consume_attributes('firezone' => { 'wireguard' => { 'endpoint' => egress_ip } })