Skip to content

Commit 67e626a

Browse files
Copilotkurowski
andcommitted
Use pam_umask instead of /etc/login.defs for umask setting
Co-authored-by: kurowski <7466+kurowski@users.noreply.github.com>
1 parent b86d5f6 commit 67e626a

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

Dockerfile

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,9 @@
11
FROM mcr.microsoft.com/devcontainers/php:8.3
22

33
# Change default umask and add user to web group so we can share write permission on web files
4-
RUN sed -i 's/^UMASK[[:space:]]*022/UMASK\t\t002/' /etc/login.defs
4+
# Configure pam_umask to set umask to 002 (works regardless of /etc/login.defs content)
5+
RUN sed -i 's/session optional\s*pam_umask.so$/session optional\t\t\tpam_umask.so umask=002/' /etc/pam.d/common-session \
6+
&& sed -i 's/session optional\s*pam_umask.so$/session optional\t\t\tpam_umask.so umask=002/' /etc/pam.d/common-session-noninteractive
57
RUN usermod -aG www-data vscode
68

79
# Add glow for formatting command usage output (and because it's just nice)

0 commit comments

Comments
 (0)