Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Include proxy env variables in git operations if set #1556

Merged
merged 3 commits into from
Dec 3, 2018
Merged

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Nov 25, 2018

Fixes #1535.

Includes proxy env variables (http_proxy, https_proxy and no_proxy) in Git operations if they are set.

For now the allowed env variables are only those required by git to
work behind a HTTP(S) proxy (`HTTP_PROXY, `HTTPS_PROXY and `NO_PROXY`).

Signed-off-by: Hidde Beydals <hello@hidde.co>
@hiddeco hiddeco requested a review from squaremo December 3, 2018 14:11
Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sound -- thanks @hiddeco

@hiddeco hiddeco merged commit 5cdd314 into master Dec 3, 2018
@hiddeco hiddeco deleted the 1535-http-proxy branch December 3, 2018 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants