Skip to content

fix: close remaining SCP action gaps and expire cached Wiz webhook se… - #16

Merged
DustyStudy merged 1 commit into
mainfrom
fix/security-review-findings
Sep 20, 2026
Merged

DustyStudy merged 1 commit into
mainfrom
fix/security-review-findings

Conversation

@DustyStudy

Copy link
Copy Markdown
Owner

…cret

  • scp-guardrails: deny-disable-security-services now also denies the current-name GuardDuty/Security Hub administrator-disassociation actions (the legacy ...FromMasterAccount names are distinct IAM actions), guardduty:DeleteMembers/StopMonitoringMembers and securityhub:BatchDisableStandards/DeleteMembers/DisassociateMembers/ DisableOrganizationAdminAccount. Applied to the policy JSON, the CloudFormation template and the Terraform module; README documents the new break-glass implication.
  • wiz-finding-bridge: the webhook secret cache now expires after SECRET_CACHE_TTL_SECONDS (default 300), so a rotated token stops being accepted within minutes instead of persisting in warm execution environments; a failed refresh no longer keeps serving the stale value.

…cret

- scp-guardrails: deny-disable-security-services now also denies the
  current-name GuardDuty/Security Hub administrator-disassociation actions
  (the legacy ...FromMasterAccount names are distinct IAM actions),
  guardduty:DeleteMembers/StopMonitoringMembers and
  securityhub:BatchDisableStandards/DeleteMembers/DisassociateMembers/
  DisableOrganizationAdminAccount. Applied to the policy JSON, the
  CloudFormation template and the Terraform module; README documents the
  new break-glass implication.
- wiz-finding-bridge: the webhook secret cache now expires after
  SECRET_CACHE_TTL_SECONDS (default 300), so a rotated token stops being
  accepted within minutes instead of persisting in warm execution
  environments; a failed refresh no longer keeps serving the stale value.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@DustyStudy
DustyStudy merged commit 3c83e34 into main Sep 20, 2026
3 checks passed
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