christianlight/tutor
6
0
Fork 0
mirror of https://github.com/ChristianLight/tutor.git synced 2024-12-13 14:43:03 +00:00
Code Issues Releases Activity

Minor simplification for checking root usage

Browse Source
This commit is contained in:
Régis Behmo 2020-11-16 12:17:15 +01:00
parent d351ca9184
commit 21231a15d0
2 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tutor/commands/cli.py
View File

@ -54,7 +54,7 @@ def main():
)
@click.pass_context
def cli(context, root):
if utils.get_user_id() == 0:
if utils.is_root():
fmt.echo_alert(
"You are running Tutor as root. This is strongly not recommended. If you are doing this in order to access"
" the Docker daemon, you should instead add your user to the 'docker' group. (see https://docs.docker.com"

7
tutor/utils.py
View File

@ -126,6 +126,13 @@ def walk_files(path):
yield os.path.join(dirpath, filename)
def is_root():
"""
Check whether tutor is being run as root/sudo.
"""
return get_user_id() == 0
def get_user_id():
"""
Portable way to get user ID. Note: I have no idea if it actually works on windows...