Skip to content

Change uid of www-data user before starting apache#286

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.

Sign up for GitHub

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

Closed
Vampouille wants to merge 3 commits into from
Closed

Change uid of www-data user before starting apache #286

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2b6cde3
Change uid of www-data user before starting apache
Vampouille Aug 8, 2016
c4dd2ec
Separate UID change in dedicated shell script
Vampouille Aug 17, 2016
3d9de66
Move apache2-foreground command from ENTRYPOINT to CMD
Vampouille Aug 17, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion apache-Dockerfile-block-2
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@

COPY docker-entrypoint.sh /docker-entrypoint.sh
COPY docker-entrypoint.d /docker-entrypoint.d

COPY apache2-foreground /usr/local/bin/
WORKDIR /var/www/html

EXPOSE 80
CMD ["apache2-foreground"]

ENTRYPOINT [ "/docker-entrypoint.sh" ]

CMD [ "apache2-foreground" ]
12 changes: 12 additions & 0 deletions docker-entrypoint.d/00-update-uid
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
#!/usr/bin/env bash

# current www-data UID
CURRENT_UID=`id -u www-data`
WANTED_UID=${RUN_AS_UID:-33}

# check if UID need to be updated
if [ $CURRENT_UID -ne $WANTED_UID ]; then

@timstallmanntimstallmannSep 27, 2016

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried using this same code in a different docker image to change the www-data UID and needed to tweak this line to if [ "$CURRENT_UID" -ne "$WANTED_UID" ]; then, and the above to WANTED_UID=${RUN_AS_UID}

@VampouilleVampouilleOct 6, 2016

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CURRENT_UID and WANTED_UID should not be empty because :

  • id command always return numeric identifier
  • ${RUN_AS_UID:-33} use 33 as default value if RUN_AS_UID is not set

Can you explain steps to reproduce your issue, maybe your www-data default uid was not set to 33.

@timstallmanntimstallmannDec 5, 2016

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Vampouille that makes sense.. I'm not clear why the ID was not set by default on my instance, but based on what you're saying it doesn't seem like this change needs to be made. Thanks for taking the time to respond!

echo "Updating www-data UID to $WANTED_UID"
usermod --non-unique --uid $WANTED_UID www-data
chown -R www-data /run/apache2 /run/lock/apache2 /var/cache/apache2/mod_cache_disk /var/log/apache2
fi
10 changes: 10 additions & 0 deletions docker-entrypoint.sh
View file
Open in desktop
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
#!/bin/bash

DIR=/docker-entrypoint.d

if [[ -d "$DIR" ]]
then
/bin/run-parts --verbose "$DIR"
fi

exec "$@"
3 changes: 3 additions & 0 deletions update.sh
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -110,6 +110,9 @@ for version in "${versions[@]}"; do
ia { ac++ }
ia && ac == 1 { system("cat '$variant'-Dockerfile-block-" ab) }
' "$base" > "$version/$target/Dockerfile"
if [ $target == "apache" ]; then
cp -vr docker-entrypoint.sh docker-entrypoint.d "$version/$target/"
fi
cp -v docker-php-ext-* "$version/$target/"
cp -v docker-php-source "$version/$target/"
dockerfiles+=( "$version/$target/Dockerfile" )
Expand Down