Skip to content

Dotfiles break in some proxy environments #13682

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

Closed
mrzarquon opened this issue Oct 7, 2022 · 3 comments
Closed

Dotfiles break in some proxy environments #13682

mrzarquon opened this issue Oct 7, 2022 · 3 comments
Labels
meta: stale This issue/PR is stale and will be closed soon type: bug Something isn't working

Comments

@mrzarquon
Copy link
Contributor

mrzarquon commented Oct 7, 2022

Bug description

tl,dr; we don't honor no_proxy when cloning dotfiles.

Supervisor copies https_proxy and http_proxy into the git clone subroutine, but ignores the no_proxy value.

For users whose SCM instances that host their dotfiles repo are behind the proxies, this is not a problem. But for users whose scm instances aren't behind the proxy and are on the no_proxy list, the dotfiles clone will fail because we explicitly create an empty environment and only set certain variables for it.

The result is the user can clone the repository once their workspace has launched, but the dotfiles won't clone during workspace launch.

This will cause significant issues unless addressed before self-hosted supports https_proxy

Steps to reproduce

Set a user environment variable setting a proxy, also set the no_proxy escaping your-github.com.

Attempt to launch a workspace with your user settings for dotfiles at your-github.com/me/dotfiles

Workspace affected

No response

Expected behavior

Wherever we apply https_proxy environment variables, we must check and apply no_proxy environment variables.

Example repository

No response

Anything else?

No response

Front logo Front conversations

@mrzarquon
Copy link
Contributor Author

This isn't fixed by #13959, it just prevents that from happening in some cases. The problem still exists.

@mrzarquon mrzarquon reopened this Oct 19, 2022
@sagor999 sagor999 moved this from Awaiting Deployment to In Validation in 🌌 Workspace Team Oct 19, 2022
@sagor999 sagor999 moved this from In Validation to Breakdown in 🌌 Workspace Team Oct 19, 2022
@subinamathew
Copy link

Do you have a lead time on this issue?

@stale
Copy link

stale bot commented Feb 19, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Feb 19, 2023
@stale stale bot closed this as completed Mar 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta: stale This issue/PR is stale and will be closed soon type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants