paskal / tg-spam / 24961436304

83%

master: 79%

Build Type

push

github

Committed by paskal

Commit Message

add WARN on InstanceID divergence + regression test for empty-blob branch

Address review:

- log a [WARN] when the persisted blob carries a non-empty InstanceID
  that differs from the CLI/env value. Behaviour is unchanged (blob
  still wins) but the divergence shifts the runtime gid and the
  Argon2 salt derived in app/config/crypt.go, so the next save would
  re-encrypt sensitive fields under a different key. Surfacing it
  beats silent.

- TestLoadConfigFromDB_PreservesCLIInstanceIDOnEmptyBlob locks in the
  new branch: stages a row whose gid matches the CLI value but whose
  JSON blob has instance_id="", then asserts loadConfigFromDB retains
  the CLI value (without the fix it would be "").

Coverage Stats

12 of 15 new or added lines in 1 file covered. (80.0%)

1 existing line in 1 file now uncovered.

8174 of 9839 relevant lines covered (83.08%)

231.49 hits per line