Skip to content

[CNAPP-24185] : BE | Public Disk access still coming up need to confirm on check #7

Merged
Chirag-CG merged 2 commits intoaccuknox:masterfrom
Vickydew1:master
Jan 28, 2026
Merged

[CNAPP-24185] : BE | Public Disk access still coming up need to confirm on check #7
Chirag-CG merged 2 commits intoaccuknox:masterfrom
Vickydew1:master

Conversation

@Vickydew1
Copy link

Used publicNetworkAccess as the authoritative signal for disk exposure. Previously, disks with networkAccessPolicy=AllowAll were flagged even when public access was disabled in Azure.

@Vickydew1 Vickydew1 requested review from Chirag-CG, NikhilRawat014 and Om-Accuknox and removed request for Om-Accuknox December 17, 2025 06:27
@Vickydew1 Vickydew1 self-assigned this Dec 17, 2025
@Vickydew1 Vickydew1 closed this Dec 17, 2025
@Vickydew1 Vickydew1 reopened this Dec 17, 2025
@Vickydew1
Copy link
Author

After confirming with the DevOps team, we now treat publicNetworkAccess as enabled if the field is missing in the resource JSON.

@Chirag-CG Chirag-CG merged commit ea0c52b into accuknox:master Jan 28, 2026
0 of 2 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.

2 participants