Match echoed condition keys in CI alert acknowledgements

ClaudiuSchuster commented

2026-10-26 21:15:27 +00:00

Owner

The receiver returns an HTTP202 acknowledgement containing the submitted condition key, so a single static JSON body cannot match acknowledgements for different conditions. Add an explicit exact-echo mode: combine the privately bound static response fields with the generated deduplication key, then require an exact, strict response match. Existing empty and fixed-JSON modes remain unchanged.

Validation: 168 tests pass. Actual loopback HTTP202 and collector tests cover two distinct keys, lost acknowledgements, retries and isolated fault/recovery; malformed, missing, extra, duplicate and mismatched fields fail closed. Independent correction review passed:168 tests,44 repeated calls with unchanged config,34 ACK and19 config countercases, plus all three collector chains. No real receiver call, credential access or monitor activation is performed by this change.

The receiver returns an HTTP202 acknowledgement containing the submitted condition key, so a single static JSON body cannot match acknowledgements for different conditions. Add an explicit exact-echo mode: combine the privately bound static response fields with the generated deduplication key, then require an exact, strict response match. Existing empty and fixed-JSON modes remain unchanged. Validation: 168 tests pass. Actual loopback HTTP202 and collector tests cover two distinct keys, lost acknowledgements, retries and isolated fault/recovery; malformed, missing, extra, duplicate and mismatched fields fail closed. Independent correction review passed:168 tests,44 repeated calls with unchanged config,34 ACK and19 config countercases, plus all three collector chains. No real receiver call, credential access or monitor activation is performed by this change.

ClaudiuSchuster added 1 commit

2026-10-26 21:15:27 +00:00

fix(watchdog): verify exact deduplication-key echo acknowledgements

access-validation-contract / contract (pull_request) Successful in 3m17s

Details Explain this check

ci / ci (pull_request) Successful in 5m24s

Details Explain this check

c8aa5514e5

ClaudiuSchuster merged commit 708c16d7dd into main

2026-10-26 21:32:44 +00:00

ClaudiuSchuster deleted branch oss-oo/ci-health-echo-ack-20261026-e3a447f4

2026-10-26 21:32:44 +00:00