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

Run gtt as a non priveleged user in docker container #117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juanluisbaptiste
Copy link

@juanluisbaptiste juanluisbaptiste commented Jul 1, 2020

Hi,

This PR fixes issue #104 , as PR #108 does not correctly set the volumes to access the gtt configuration file without having to mount the complete user home directory which is a security risk. It also uses a non privileged user to run gtt instead of the root user, which is also a security risk.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 65.979% when pulling 6d1ece0 on juanluisbaptiste:run_as_non_root_user into ec5ca47 on kriskbx:master.

@juanluisbaptiste juanluisbaptiste changed the title Run gtt as a non priveleged user Run gtt as a non priveleged user in docker container Jul 1, 2020
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.

2 participants