Skip to content

Commit

Permalink
main: Write output ignition files with mode 0600
Browse files Browse the repository at this point in the history
Because the config is commonly expected to have secret values, use mode 0600.
xref coreos/fedora-coreos-docs#306
  • Loading branch information
cgwalters committed Aug 27, 2021
1 parent 97c6866 commit 84041b5
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
4 changes: 3 additions & 1 deletion internal/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,9 @@ func main() {

if output != "" {
var err error
outfile, err = os.OpenFile(output, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)
// Because the config is commonly expected to have secret values, use mode 0600.
// xref https://github.com/coreos/fedora-coreos-docs/issues/306
outfile, err = os.OpenFile(output, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0600)
if err != nil {
fail("failed to open %s: %v\n", output, err)
}
Expand Down
17 changes: 17 additions & 0 deletions test
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,23 @@ trap 'rm -r tmpdocs' EXIT
mkdir -p tmpdocs/files-dir/tree
touch tmpdocs/files-dir/{config.ign,ca.pem,file,file-epilogue,local-file3}

tmpd=$(mktemp -d)
${BIN_PATH}/${NAME} --strict --output ${tmpd}/foo.ign << 'EOF'
variant: fcos
version: 1.1.0
storage:
files:
- path: /etc/somesecret
mode: 0600
contents:
source: https://example.com/etc/somesecret
EOF
m=$(stat -c '%a' ${tmpd}/foo.ign)
if test "${m}" != 600; then
echo "Unexpected ignition mode: ${m}" 1>&2
exit 1
fi

for doc in docs/*md
do
echo "Checking $doc"
Expand Down

0 comments on commit 84041b5

Please sign in to comment.