Merge pull request #25886 from hashicorp/b/crash-webpubsub-25855

`webpubsub`: fixing a panic where `networkACL.PublicNetwork.Deny` was nil
hashicorp · May 8, 2024 · bb28b69 · bb28b69
2 parents 2b69305 + 1054311
commit bb28b69
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/internal/services/signalr/web_pubsub_network_acl_resource.go b/internal/services/signalr/web_pubsub_network_acl_resource.go
@@ -165,7 +165,7 @@ func resourceWebPubsubNetworkACLCreateUpdate(d *pluginsdk.ResourceData, meta int
 
 	if defaultAction == webpubsub.ACLActionAllow && networkACL.PublicNetwork.Allow != nil && len(*networkACL.PublicNetwork.Allow) != 0 {
 		return fmt.Errorf("when `default_action` is `Allow` for `public_network`, `allowed_request_types` cannot be specified")
-	} else if defaultAction == webpubsub.ACLActionDeny && len(*networkACL.PublicNetwork.Deny) != 0 {
+	} else if defaultAction == webpubsub.ACLActionDeny && networkACL.PublicNetwork.Deny != nil && len(*networkACL.PublicNetwork.Deny) != 0 {
 		return fmt.Errorf("when `default_action` is `Deny` for `public_network`, `denied_request_types` cannot be specified")
 	}