Skip to content

feat(scanner): add AZ-NET-014 VNet peering gateway transit rule#94

Merged
Vishnu2707 merged 1 commit into
openshield-org:devfrom
aav-wh:feat/az-net-014
May 30, 2026
Merged

feat(scanner): add AZ-NET-014 VNet peering gateway transit rule#94
Vishnu2707 merged 1 commit into
openshield-org:devfrom
aav-wh:feat/az-net-014

Conversation

@aav-wh
Copy link
Copy Markdown
Contributor

@aav-wh aav-wh commented May 30, 2026

Summary

Adds a new CSPM scanner rule to detect Virtual Network peering connections with gateway transit enabled, which can break network segmentation and allow lateral movement between isolated network zones.

Changes

  • scanner/rules/az_net_014.py — detects peerings with allowGatewayTransit or useRemoteGateways enabled
  • scanner/azure_client.py — adds get_vnet_peerings() and get_azure_firewalls() methods
  • playbooks/cli/fix_az_net_014.sh — CLI remediation script to disable gateway transit
  • All 4 compliance framework JSONs updated with AZ-NET-014 mappings (CIS 6.4, NIST PR.AC-5, ISO A.13.1.1, SOC2 CC6.6)

Test

  • python -m py_compile scanner/rules/az_net_014.py
  • python -m py_compile scanner/azure_client.py
  • All 4 compliance JSONs validated ✅

Related Issue

Closes #93

- Add scanner/rules/az_net_014.py to detect VNet peerings with
  allowGatewayTransit or useRemoteGateways enabled
- Add get_vnet_peerings() and get_azure_firewalls() to azure_client.py
- Add playbooks/cli/fix_az_net_014.sh remediation script
- Update all 4 compliance framework JSONs with AZ-NET-014 mappings
Copy link
Copy Markdown
Member

@Vishnu2707 Vishnu2707 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Rule logic is correct, get_vnet_peerings() follows the existing client pattern, compliance mappings are additive-only, remediation script is safe with the production warning. Good to merge.

@Vishnu2707 Vishnu2707 merged commit 3636dd7 into openshield-org:dev May 30, 2026
1 check 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.

2 participants