[trafficserver] branch master updated: Disable proxies when running autest (#9756)

[rr...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

rrm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 37a58127d0 Disable proxies when running autest (#9756)
37a58127d0 is described below

commit 37a58127d049709119eb8e58b4a938fc2b504d9a
Author: midchildan <gi...@midchildan.org>
AuthorDate: Sat Jul 1 01:12:10 2023 +0900

    Disable proxies when running autest (#9756)
    
    When HTTP proxies are enabled on the system, autest would fail to run properly
    because requests should never leave localhost. Now, proxies can be disabled by
    unsetting the relevant environment variables in the CLI. However, it's not ideal
    in this case because the autest script uses pipenv, which fetches Python
    packages online, to run the actual tests.
    
    This change fixes that by unsetting the environment variables after pipenv is
    finished.
    
    Co-authored-by: Brian Neradt <br...@gmail.com>
---
 tests/autest.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/autest.sh b/tests/autest.sh
index 83658bc41a..048c160c42 100755
--- a/tests/autest.sh
+++ b/tests/autest.sh
@@ -33,4 +33,6 @@ export PYTHONPATH=$(pwd):$PYTHONPATH
 ./test-env-check.sh || fail "Failed Python environment checks."
 # this is for rhel or centos systems
 echo "Environment config finished. Running AuTest..."
-exec pipenv run autest "$@" -D gold_tests
+exec pipenv run env \
+    HTTP_PROXY= HTTPS_PROXY= NO_PROXY= http_proxy= https_proxy= no_proxy= \
+    autest -D gold_tests "$@"