Skip to content

[codex] Respect unknown org membership in cross-account trust#68

Merged
InfoSecHack merged 1 commit into
mainfrom
codex/cross-account-trust-org-membership-status
Jun 5, 2026
Merged

[codex] Respect unknown org membership in cross-account trust#68
InfoSecHack merged 1 commit into
mainfrom
codex/cross-account-trust-org-membership-status

Conversation

@InfoSecHack
Copy link
Copy Markdown
Owner

Summary

  • Updates cross_account_trust to prefer explicit org_membership_status over legacy org_member booleans.
  • Keeps member sources on the existing same-org severity downgrade path.
  • Keeps non-member sources on the confirmed external path.
  • Makes unknown org membership visible in trace, title, exit reason, and finding assumptions without wording it as confirmed external.
  • Adds pipeline-shaped tests using parser output plus _run_resolution and FactGraph.

Root Cause

After tri-state synthetic principal metadata was added, the cross-account trust reasoner still consumed legacy org_member semantics and used binary title wording. Unknown org membership could therefore be reported as an external cross-account trust in the finding title even though the verdict path was already conservative.

Validation

  • python -m pytest -q tests/integration/test_cross_account_org_membership_status.py tests/test_cross_account_reasoner.py tests/test_cross_account.py tests/resolver/test_org_membership_uncertainty.py tests/integration/test_full_pipeline_reasoner_verdicts.py tests/test_golden_findings.py
  • ./scripts/check.sh
  • ./scripts/test_fast.sh
  • git diff --cached --check
  • account/ARN hygiene scans
  • Terraform/raw artifact scan

@InfoSecHack InfoSecHack marked this pull request as ready for review June 5, 2026 21:51
@InfoSecHack InfoSecHack merged commit 8436b73 into main Jun 5, 2026
6 checks passed
@InfoSecHack InfoSecHack deleted the codex/cross-account-trust-org-membership-status branch June 5, 2026 21:51
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