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

Multi-user installation + proxy must use sudo -E #139

Merged
merged 1 commit into from Jan 30, 2013

Conversation

Projects
None yet
2 participants
@nilbus
Contributor

nilbus commented Jan 30, 2013

Otherwise, sudo resets the environment and loses the http_proxy environment variable.

Multi-user installation + proxy must use sudo -E
Otherwise, sudo resets the environment and loses the http_proxy environment variable.
@mpapis

This comment has been minimized.

Show comment
Hide comment
@mpapis

mpapis Jan 30, 2013

Member

why not set the proxy on machine level in /etc/curlrc?

Member

mpapis commented Jan 30, 2013

why not set the proxy on machine level in /etc/curlrc?

@nilbus

This comment has been minimized.

Show comment
Hide comment
@nilbus

nilbus Jan 30, 2013

Contributor

At least for my proxy setup, users must authenticate to the proxy with
their own user credentials. This can't be set system-wide.

Contributor

nilbus commented Jan 30, 2013

At least for my proxy setup, users must authenticate to the proxy with
their own user credentials. This can't be set system-wide.

mpapis added a commit that referenced this pull request Jan 30, 2013

Merge pull request #139 from nilbus/patch-1
Multi-user installation + proxy must use sudo -E

@mpapis mpapis merged commit ee67c53 into rvm:master Jan 30, 2013

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment