Skip to content

Commit c957acc

Browse files
committed
fix: update #nosec comments for security annotations across multiple …
1 parent a701870 commit c957acc

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

docker/worker/Dockerfile

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,15 @@ RUN apk --no-cache add docker-cli ca-certificates
4343
# user: "${WORKER_UID:-1000}:${DOCKER_GID:-999}".
4444
#
4545
# UR bug_021: DOCKER_GID=0 (macOS Docker Desktop часто использует root group)
46-
# конфликтует с уже существующей root-группой alpine. Спец-кейс: если GID=0,
47-
# просто добавляем appuser в существующий root; иначе создаём новую docker-группу.
46+
# конфликтует с уже существующей root-группой alpine. В Alpine 3.20 GID 999
47+
# занят группой `ping`. Логика: если группа с нужным GID уже существует —
48+
# переиспользуем её; иначе создаём docker-группу.
4849
RUN addgroup -g 1000 appuser && \
4950
adduser -D -u 1000 -G appuser appuser && \
50-
if [ "${DOCKER_GID}" = "0" ]; then \
51-
adduser appuser root; \
51+
existing_group=$(awk -F: -v gid="${DOCKER_GID}" '$3==gid {print $1; exit}' /etc/group) && \
52+
if [ -n "$existing_group" ]; then \
53+
adduser appuser "$existing_group"; \
5254
else \
53-
(delgroup docker 2>/dev/null || true) && \
5455
addgroup -g ${DOCKER_GID} docker && \
5556
adduser appuser docker; \
5657
fi

0 commit comments

Comments
 (0)