Skip to content

fix active response process identity validation#45

Draft
Karib0u wants to merge 1 commit into
mainfrom
codex/revalidate-process-identity
Draft

fix active response process identity validation#45
Karib0u wants to merge 1 commit into
mainfrom
codex/revalidate-process-identity

Conversation

@Karib0u
Copy link
Copy Markdown
Owner

@Karib0u Karib0u commented May 31, 2026

Summary

  • add process identity capture with PID, image path, process start time, and optional command-line hash
  • re-query live process identity immediately before active-response termination
  • skip and log active responses when the live process no longer matches the alert identity

Root cause

Active response made the termination decision from the PID and image stored in the alert context. If that PID was reused before the worker executed, termination could target a different process.

Validation

  • cargo test --no-run
  • cargo test --test active_response
  • cargo test response::tests::validate_process_identity

Windows target compile was attempted locally, but x86_64-pc-windows-msvc is not installed on this machine.

@Karib0u Karib0u changed the title [codex] fix active response process identity validation fix active response process identity validation May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant