FROSADO / onecloudriver / 31941645570

77%

main: 78%

Build Type

Pull #86

github

Committed by FROSADO

Commit Message

fix(mount): make --cache-dir a session-only override (#85)

A one-off `mount --cache-dir /tmp/...` permanently overwrote the
account JSON (`acc.Mount.CacheDir`), so future mounts (e.g. systemd
service restarts) pointed at a temporary path — which, once deleted,
crash-looped the services until the config was manually restored.

- mount.go: track whether --cache-dir came from the flag and exclude
  it from the persisted config (new buildPersistedMountConfig helper);
  print "(session only, not saved to account config)" when used.
- All other flag overrides (cache-ttl, delta-interval, ...) are safe
  scalars and keep being persisted as before.
- Tests: buildPersistedMountConfig keeps the previous CacheDir when
  the flag is given (also when the persisted value is empty) and
  persists all other fields; persists CacheDir when no flag is set.
- Docs: flag help, command Long text, MANUAL.md/.es, ARCHITECTURE*
  now document the session-only semantics.

Generated with Codebuff 🤖
Co-Authored-By: Codebuff <noreply@codebuff.com>

Pull Request Pull Request #86: fix(mount): make --cache-dir a session-only override (#85)

Coverage Stats

15 of 27 new or added lines in 1 file covered. (55.56%)

2 existing lines in 1 file now uncovered.

3848 of 4976 relevant lines covered (77.33%)

479.69 hits per line