-
Notifications
You must be signed in to change notification settings - Fork 331
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
Move $HOME env variable outside web server's Document root #454
base: master
Are you sure you want to change the base?
Conversation
[test] |
[test-all] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this pull request. The change itself LGTM. Let's wait for the tests. Good Job.
Pull Request validationFailed🔴 Failed or pending statuses - Success🟢 Review - Reviewed by undefined |
[test-all] |
Testing Farm results
|
Overwrite $HOME env variable for all supported versions. Default $HOME is directed inside the web server's Document root, which means that applications that save potentially private data to $HOME (e.g. bash's histfile) will save them into a folder accessible outside the container via the web server. This means there is a possibility of leaking the data.
This does not occur in all cases, namely bash won't create a histfile at all, when user sets a different user via
--user=
argument inpodman run
command, as in that case bash doesn't have a permission to write into the $HOME.Fixes: #255