Skip to content

Commit 055ea5b

Browse files
authored
Merge pull request #7 from openziti/fix-action
chmod the inner build script so the non-root container user can read it
2 parents f4d03a0 + 75df561 commit 055ea5b

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

tools/build-sdk.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,13 @@ mkdir -p /out
9696
find bin/ -name "*.ipk" -print -exec cp {} /out/ \;
9797
EOF
9898

99+
# mktemp creates the file mode 0600 owned by the host user. The SDK image
100+
# runs the inner script as the unprivileged 'builder' user (uid 1000), and
101+
# on Linux Docker the host uid is preserved -- so a 0600 file owned by
102+
# host uid 1001 (typical CI runner) is unreadable to container uid 1000.
103+
# Make it world-readable. The contents are not secret.
104+
chmod 0644 "$TMP_INNER"
105+
99106
echo "==> Running build in container"
100107
docker run --rm \
101108
-v "$FEED_PATH:/feed:ro" \

0 commit comments

Comments
 (0)