fix(web): log settingsupdate validation/apply failures server-side #68

Merged

David merged 1 commit from fix/VS-44-log-settingsupdate-failures into main

2026-06-21 18:27:08 +02:00

David commented

2026-06-21 18:17:41 +02:00

Owner

A failed settingsupdate control action folded its error into the client ack but wrote nothing to the server log, so the failure was invisible in docker logs. Emit a warn! naming the section and reason in the ControlIn::SettingsUpdate Err arm, and a debug! for the not-authorized rejection. The value payload is never logged (it can carry secrets such as fetch token / HTTP-auth password); only the secret-free section and apply_* error string are logged. Ack behavior is unchanged.

#VS-44

A failed `settingsupdate` control action folded its error into the client ack but wrote nothing to the server log, so the failure was invisible in `docker logs`. Emit a `warn!` naming the section and reason in the `ControlIn::SettingsUpdate` Err arm, and a `debug!` for the `not-authorized` rejection. The `value` payload is never logged (it can carry secrets such as fetch `token` / HTTP-auth `password`); only the secret-free `section` and `apply_*` error string are logged. Ack behavior is unchanged. #VS-44

David added 1 commit

2026-06-21 18:17:41 +02:00

fix(web): log settingsupdate validation/apply failures server-side

Check / fmt + clippy + build + tests (pull_request) Successful in 2m3s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

6d2cf229d8

A failed `settingsupdate` control action folded its error into the client ack but wrote nothing to the server log, so the failure was invisible in `docker logs`. Emit a `warn!` naming the section and reason in the `ControlIn::SettingsUpdate` Err arm, and a `debug!` for the `not-authorized` rejection. The `value` payload is never logged (it can carry secrets such as fetch `token` / HTTP-auth `password`); only the secret-free `section` and `apply_*` error string are logged. Ack behavior is unchanged.

#VS-44