paskal / spot / 28754879180

84%

master: 84%

Build Type

push

github

Committed by paskal

Commit Message

guards: decrypt length check, ssh user nil guard, command count across hosts

- decrypt now rejects ciphertext shorter than the 24-byte nonce plus
  16-byte salt with an error instead of panicking on the slice.
- sshUser had an inverted nil guard: when pbook was nil it still
  dereferenced pbook.User. Only use the playbook user when pbook is
  non-nil.
- ProcResp.Commands counted only host index 0, undercounting when that
  host skips or fails commands the others run. Report the max command
  count across hosts instead, tracked under the existing lock. A new
  two-host test (only_on skipping a command on the index-0 host) asserts
  the fuller host's count is reported.

Coverage Stats

10 of 10 new or added lines in 3 files covered. (100.0%)

2 existing lines in 1 file now uncovered.

3195 of 3801 relevant lines covered (84.06%)

59.18 hits per line