packages/standard/ami/ami-configure.sh:21:
mkdir /mnt/docker
# TODO: wait, chown? what is user 711? is that supposed to be chmod?
chown 711 /mnt/docker
The existing TODO captures the confusion. chown 711 sets owner to UID 711, which doesn't correspond to any OpenEMR image user I can find. chmod 711 (world-executable-only, owner rwx) would make more sense for a Docker data directory — containers need traversal but not other users listing it.
Need to verify against the historical intent (blame / commit history), decide the right form, and update. Flagged on PR #653 (review thread).
packages/standard/ami/ami-configure.sh:21:mkdir /mnt/docker # TODO: wait, chown? what is user 711? is that supposed to be chmod? chown 711 /mnt/dockerThe existing TODO captures the confusion.
chown 711sets owner to UID 711, which doesn't correspond to any OpenEMR image user I can find.chmod 711(world-executable-only, owner rwx) would make more sense for a Docker data directory — containers need traversal but not other users listing it.Need to verify against the historical intent (blame / commit history), decide the right form, and update. Flagged on PR #653 (review thread).