Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Do we need to really run this as root? #2

Open
nolim1t opened this issue Nov 5, 2021 · 2 comments
Open

Do we need to really run this as root? #2

nolim1t opened this issue Nov 5, 2021 · 2 comments

Comments

@nolim1t
Copy link
Contributor

nolim1t commented Nov 5, 2021

Do we need to really run this as root?

Some people might get freaked out about this. Should at least do sudo in the individual commands that require it (and inform the user).

if os.getuid() != 0:

@AaronDewes
Copy link
Member

Do we need to really run this as root?

Currently, yes. We manually install docker compose v2 (a docker CLI plugin, not a separate tool) to the users home directory and all other scripts run as root. But I'll keep this on the roadmap.

@AaronDewes AaronDewes pinned this issue Nov 6, 2021
@nolim1t
Copy link
Contributor Author

nolim1t commented Nov 7, 2021

Do we need to really run this as root?

Currently, yes. We manually install docker compose v2 (a docker CLI plugin, not a separate tool) to the users home directory and all other scripts run as root. But I'll keep this on the roadmap.

awesome would be good to install docker-compose v2 CLI to whatever the default user is eventually. Some people may get nervous about using root and its also a potential attack vector

@AaronDewes AaronDewes moved this to 📋 Backlog in Citadel development May 31, 2022
eldios added a commit to eldios/core that referenced this issue Jul 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants