Skip to content

Commit

Permalink
Add ALL_PROXY and NO_PROXY to collected env_vars
Browse files Browse the repository at this point in the history
Resolves: #3788

Signed-off-by: Pablo Fernández Rodríguez<[email protected]>
  • Loading branch information
pafernanr committed Sep 28, 2024
1 parent 60356d6 commit 83572e0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
7 changes: 6 additions & 1 deletion sos/report/plugins/foreman.py
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,12 @@ def collect_proxies(self):
timeout=10)

# collect http[|s]_proxy env.variables
self.add_env_var(["http_proxy", "https_proxy"])
self.add_env_var([
'HTTP_PROXY',
'HTTPS_PROXY',
'NO_PROXY',
'ALL_PROXY',
])

def build_query_cmd(self, query, csv=False, binary="psql"):
"""
Expand Down
7 changes: 6 additions & 1 deletion sos/report/plugins/foreman_proxy.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,12 @@ def setup(self):
])

# collect http[|s]_proxy env.variables
self.add_env_var(["http_proxy", "https_proxy"])
self.add_env_var([
'HTTP_PROXY',
'HTTPS_PROXY',
'NO_PROXY',
'ALL_PROXY',
])

def postproc(self):
self.do_path_regex_sub(
Expand Down

0 comments on commit 83572e0

Please sign in to comment.