export silently overwrites an existing output file #628

New issue

Open

opened 2026-08-10 11:42:38 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-08-10 11:42:38 +00:00

Owner

tpagectl export SITE writes via os.Create(output) in internal/tpagectl/export/export.go, silently truncating an existing <SITE>.yaml (or --output target). Because these exported documents are meant to be reviewed and edited (and later consumed by apply), a re-export destroys any edits made since the last export.

Refuse to overwrite an existing file (or require an explicit --force flag) so user edits aren't clobbered accidentally.

See #627

`tpagectl export SITE` writes via `os.Create(output)` in `internal/tpagectl/export/export.go`, silently truncating an existing `<SITE>.yaml` (or `--output` target). Because these exported documents are meant to be reviewed and edited (and later consumed by `apply`), a re-export destroys any edits made since the last export. Refuse to overwrite an existing file (or require an explicit `--force` flag) so user edits aren't clobbered accidentally. See https://git.lan.thwap.org/thwap/thwap-pagesd/pulls/627

the.auditor added the

Kind/Bug

Priority

Medium

labels

2026-08-10 11:42:44 +00:00