Skip to content

Conversation

@bradhe
Copy link
Contributor

@bradhe bradhe commented Jan 13, 2026

Some users reported weird behavior when installing certain third-party libraries and using them with Tower. Turns out, they expected to have a $HOME env var to save some downloaded content to. We jettisoned that env var while scrubbing the environment for other secrets that may be lurking.

This PR adds it and a few other important UNIX-y ones back in to Unix environments.

@bradhe bradhe requested review from sammuti and socksy January 13, 2026 20:58
Copy link
Contributor

@socksy socksy left a comment

Choose a reason for hiding this comment

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

I am pretty sure the $PATH one is gonna bite us in the arse soon but let's get this in to fix $HOME

@bradhe
Copy link
Contributor Author

bradhe commented Jan 14, 2026

I agree re: $PATH. We want a scrubbed/default version of $PATH but I couldn't really think of how to get it at the time.

@bradhe bradhe merged commit 28709cf into develop Jan 14, 2026
5 checks passed
@bradhe bradhe deleted the tasks/add-home-env-var-to-child-processes branch January 14, 2026 10:18
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.

3 participants