Skip to content

Commit da38dbe

Browse files
committed
fixed issue #400
1 parent 70bc26e commit da38dbe

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

internal/scanners/dbw/rules.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import (
77
"strings"
88

99
"github.com/Azure/azqr/internal/scanners"
10-
"github.com/Azure/azqr/internal/to"
1110
"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/databricks/armdatabricks"
1211
)
1312

@@ -73,8 +72,13 @@ func (a *DatabricksScanner) GetRecommendations() map[string]scanners.AzqrRecomme
7372
Impact: scanners.ImpactMedium,
7473
Eval: func(target interface{}, scanContext *scanners.ScanContext) (bool, string) {
7574
c := target.(*armdatabricks.Workspace)
76-
broken := c.Properties.Parameters.EnableNoPublicIP != nil && c.Properties.Parameters.EnableNoPublicIP.Value == to.Ptr(true)
77-
return broken, ""
75+
ok := c.Properties != nil &&
76+
c.Properties.Parameters != nil &&
77+
c.Properties.Parameters.EnableNoPublicIP != nil &&
78+
c.Properties.Parameters.EnableNoPublicIP.Value != nil &&
79+
*c.Properties.Parameters.EnableNoPublicIP.Value
80+
81+
return !ok, ""
7882
},
7983
LearnMoreUrl: "https://learn.microsoft.com/en-us/azure/databricks/security/network/secure-cluster-connectivity",
8084
},

0 commit comments

Comments
 (0)