-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid going through HTTP proxy for requests to localhost in refine script #6361
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! The issue mentions that a similar adaptation should be done for downloads done via wget
. Could you also take care of that? According to the manual it should be enough to define a no_proxy=127.0.0.1
environment variable.
Yes Sure. I made similar changes for wget please review. |
refine
Outdated
CURL_RETURN=$? | ||
if [ $CURL_RETURN -eq "7" ] || [ $CURL_RETURN -eq "22" ] ; then | ||
NOT_RUNNING="1" | ||
fi | ||
elif [ "$WGET" ] ; then | ||
wget -O - $URL > /dev/null 2>&1 | ||
wget --noproxy 127.0.0.1 -O - $URL > /dev/null 2>&1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, the same syntax does not work for wget, as --noproxy
is not a valid wget option.
refine
Outdated
CURL_RETURN=$? | ||
if [ $CURL_RETURN -eq "7" ] || [ $CURL_RETURN -eq "22" ] ; then | ||
NOT_RUNNING="1" | ||
fi | ||
elif [ "$WGET" ] ; then | ||
wget -O - $URL > /dev/null 2>&1 | ||
wget --no-proxy -q -O - $URL_LOCAL > /dev/null 2>&1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wget --no-proxy -q -O - $URL_LOCAL > /dev/null 2>&1 | |
no_proxy=127.0.0.1 wget -O - $URL_LOCAL > /dev/null 2>&1 |
refine
Outdated
elif [ "$WGET" ] ; then | ||
RUNNING=`wget -q -O - $URL | grep "$CHECK_STR"` | ||
RUNNING=`wget --no-proxy -q -O - $URL_LOCAL | grep "$CHECK_STR"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RUNNING=`wget --no-proxy -q -O - $URL_LOCAL | grep "$CHECK_STR"` | |
RUNNING=`no_proxy=127.0.0.1 wget -O - $URL_LOCAL | grep "$CHECK_STR"` |
@wetneb I made the suggested changes, please let me know if I need to follow any steps since this is my first open-source contribution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Fixes #2000