Correct syntax

alan-turing-institute · Oct 31, 2024 · 025155a · 025155a
1 parent 60158d5
commit 025155a
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/data_safe_haven/infrastructure/components/wrapped/nfsv3_storage_account.py b/data_safe_haven/infrastructure/components/wrapped/nfsv3_storage_account.py
@@ -26,18 +26,18 @@ def __init__(
         *,
         account_name: Input[str],
         allowed_ip_addresses: Input[Sequence[str]] | None,
-        allowed_service_tag: Input[Sequence[str]] | None,
+        allowed_service_tag: AzureServiceTag | None,
         location: Input[str],
         resource_group_name: Input[str],
         subnet_id: Input[str],
         opts: ResourceOptions,
         tags: Input[Mapping[str, Input[str]]],
     ):
         if allowed_service_tag == AzureServiceTag.INTERNET:
-            default_action = storage.DefaultAction.ALLOW,
-            ip_rules = None
+            default_action = storage.DefaultAction.ALLOW
+            ip_rules = []
         else:
-            default_action = storage.DefaultAction.DENY,
+            default_action = storage.DefaultAction.DENY
             ip_rules = Output.from_input(allowed_ip_addresses).apply(
                 lambda ip_ranges: [
                     storage.IPRuleArgs(