Skip to content
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

Enable use of proxy when downloading Git binary #539

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

itsananderson
Copy link

When this project switched away from requests, support for configuring a proxy via the HTTP_PROXY/HTTPS_PROXY environment variables was lost. This change restores proxy support by adding the global-agent package as a dependency and setting it up in the download script before attempting to download Git.

By default, global-agent only looks for environment variables prefixed by GLOBAL_AGENT_ (e.g. GLOBAL_AGENT_HTTP_PROXY). To maintain backward compatibility, the download script checks for the old HTTP_PROXY/HTTPS_PROXY variables, and copies them to the GLOBAL_AGENT_ equivalent if not already defined.

Fixes #498

@custa
Copy link

custa commented Nov 15, 2023

it work
/lgtm

@panther7
Copy link

panther7 commented Aug 17, 2024

Why is not still merged?

@tidy-dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proxy support seems to have regressed
3 participants