Skip to content

Commit

Permalink
Allow data providers address to be 'Internet'
Browse files Browse the repository at this point in the history
  • Loading branch information
JimMadge committed Oct 21, 2024
1 parent 1f559b0 commit b09e683
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions data_safe_haven/config/config_sections.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ class ConfigSectionSRE(BaseModel, validate_assignment=True):
admin_email_address: EmailAddress
admin_ip_addresses: list[IpAddress] = []
databases: UniqueList[DatabaseSystem] = []
data_provider_ip_addresses: list[IpAddress] = []
data_provider_ip_addresses: list[IpAddress] | AzureServiceTag = []
remote_desktop: ConfigSubsectionRemoteDesktopOpts
research_user_ip_addresses: list[IpAddress] | AzureServiceTag = []
storage_quota_gb: ConfigSubsectionStorageQuotaGB
Expand All @@ -67,8 +67,6 @@ class ConfigSectionSRE(BaseModel, validate_assignment=True):

@field_validator(
"admin_ip_addresses",
"data_provider_ip_addresses",
# "research_user_ip_addresses",
mode="after",
)
@classmethod
Expand All @@ -81,6 +79,7 @@ def ensure_non_overlapping(cls, v: list[IpAddress]) -> list[IpAddress]:
return v

@field_validator(
"data_provider_ip_addresses",
"research_user_ip_addresses",
mode="after",
)
Expand Down

0 comments on commit b09e683

Please sign in to comment.