Skip to content

Commit

Permalink
Merge pull request NixOS#10067 from ramboman/fix-proxy-nix
Browse files Browse the repository at this point in the history
`nix`: Fix `haveInternet` to check for proxy
  • Loading branch information
thufschmitt committed Feb 23, 2024
2 parents 6a5210f + bca737d commit accae60
Showing 1 changed file with 20 additions and 0 deletions.
20 changes: 20 additions & 0 deletions src/nix/main.cc
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,24 @@ void chrootHelper(int argc, char * * argv);

namespace nix {

static bool haveProxyEnvironmentVariables()
{
static const std::vector<std::string> proxyVariables = {
"http_proxy",
"https_proxy",
"ftp_proxy",
"HTTP_PROXY",
"HTTPS_PROXY",
"FTP_PROXY"
};
for (auto & proxyVariable: proxyVariables) {
if (getEnv(proxyVariable).has_value()) {
return true;
}
}
return false;
}

/* Check if we have a non-loopback/link-local network interface. */
static bool haveInternet()
{
Expand All @@ -55,6 +73,8 @@ static bool haveInternet()
}
}

if (haveProxyEnvironmentVariables()) return true;

return false;
}

Expand Down

0 comments on commit accae60

Please sign in to comment.