-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
[BUG] UMASK ignored #169
Comments
Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid. |
This is not a valid test. |
Same result abc@5ab65b71285f:/$ /usr/bin/with-contenv env |
|
but the user not create file with the good umask |
Do you know for certain that nzbget respects the umask variable? |
If I've understood correctly, the env parameter change the shell umask of user abc. Do you agree? |
No, its independent of the user. the shell just needs to be run in the container environment context |
We just realized the umask setting in alpine 3.20 based images broke a couple of days ago and nzbget built after that so the latest has it broken. The next build of nzbget will fix it. With that said, seeing as nzbget has a umask setting in its gui, it may be ignoring the environment umask setting and using its own gui setting instead. That I'm not sure. Not every app respects the environment umask setting. Also, to clarify, the way umask is set in the container is quite convoluted and hard to confirm. It is set for the app's service in a wrapper. Exec'ing into the container and running |
A new build is pushed. With the new image, here's my quick test to check whether umask is being set correctly in the app's shell:
The second command returns |
Is there an existing issue for this?
Current Behavior
Hello
I report in the last version of docker-nzbget the umask environnement variable is ignored
Expected Behavior
The umask must be taken into account
Steps To Reproduce
Set UMASK=002 in docker-compose.yml file
docker compose exec --user abc nzbget sh
abc@5ab65b71285f:/$ env
...
PGID=1005
PUID=999
UMASK=002
TZ=Europe/Paris
...
abc@5ab65b71285f:/$ umask
0022
Environment
CPU architecture
x86-64
Docker creation
Container logs
The text was updated successfully, but these errors were encountered: