fix: use SUDO_USER for docker group to get the invoking user not root
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -46,11 +46,13 @@ if ! docker compose version >/dev/null 2>&1 && ! command -v docker-compose >/dev
|
||||
fi
|
||||
echo " [ok] docker-compose"
|
||||
|
||||
# Ensure current user is in docker group
|
||||
if ! id | grep -q '(docker)'; then
|
||||
echo " Adding $(id -un) to docker group..."
|
||||
usermod -aG docker "$(id -un)"
|
||||
# Ensure the invoking user is in the docker group (use SUDO_USER if available)
|
||||
TARGET_USER="${SUDO_USER:-$(id -un)}"
|
||||
if ! groups "$TARGET_USER" 2>/dev/null | grep -q '\bdocker\b'; then
|
||||
echo " Adding $TARGET_USER to docker group..."
|
||||
usermod -aG docker "$TARGET_USER"
|
||||
echo " NOTE: Log out and back in (or run 'newgrp docker') for docker group to take effect."
|
||||
fi
|
||||
echo " [ok] docker group ($TARGET_USER)"
|
||||
|
||||
echo "=== All dependencies OK ==="
|
||||
|
||||
Reference in New Issue
Block a user