Skip to content

Commit 66ae37d

Browse files
committed
Fix networking and database connection from API
1 parent 195e6ea commit 66ae37d

6 files changed

Lines changed: 16 additions & 10 deletions

File tree

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -357,3 +357,4 @@ MigrationBackup/
357357
*.tfstate
358358
*.tfstate.backup
359359
terraform/.terraform.lock.hcl
360+
terraform/.terraform.tfstate.lock.info

terraform/api_server.tf

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,4 @@ resource "azurerm_linux_web_app" "api_server_web_app" {
2424
type = "SQLAzure"
2525
value = local.db_connection_string
2626
}
27-
28-
virtual_network_subnet_id = azurerm_subnet.subnet.id
2927
}

terraform/database.tf

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,9 @@ resource "azurerm_mssql_database" "sqldb" {
2525
}
2626
}
2727

28-
resource "azurerm_mssql_virtual_network_rule" "sqlserver_subnet_rule" {
29-
name = "sql-vnet-rule"
30-
server_id = azurerm_mssql_server.sqlserver.id
31-
subnet_id = azurerm_subnet.subnet.id
28+
resource "azurerm_mssql_firewall_rule" "firewall_rule" {
29+
name = "AllowAllAzureServices"
30+
server_id = azurerm_mssql_server.sqlserver.id
31+
start_ip_address = "0.0.0.0"
32+
end_ip_address = "0.0.0.0"
3233
}

terraform/keyvault.tf

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,4 +38,10 @@ resource "azurerm_key_vault_secret" "sqldb_connectionstring" {
3838
name = "${local.resource_prefix}-db-connection-string"
3939
value = local.db_connection_string
4040
key_vault_id = azurerm_key_vault.key_vault.id
41-
}
41+
}
42+
43+
resource "azurerm_key_vault_secret" "sqldb_admin_password" {
44+
name = "${local.resource_prefix}-db-admin-password"
45+
value = random_password.admin_password.result
46+
key_vault_id = azurerm_key_vault.key_vault.id
47+
}

terraform/main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ resource "random_password" "admin_password" {
1414
}
1515

1616
locals {
17-
db_connection_string = "Server=tcp:${local.sql_server_name}.database.windows.net,1433;Persist Security Info=False;User ID=${random_string.admin_login.result};Password=${random_password.admin_password.result};MultipleActiveResultSets=False;Encrypt=True;TrustServerCertificate=False;Connection Timeout=30;"
17+
db_connection_string = "Server=tcp:${local.sql_server_name}.database.windows.net,1433;Initial Catalog=pocketddd-dev-sqldatabase;Persist Security Info=False;User ID=${random_string.admin_login.result};Password=${random_password.admin_password.result};MultipleActiveResultSets=False;Encrypt=True;TrustServerCertificate=False;Connection Timeout=30;"
1818
}
1919

2020
resource "azurerm_virtual_network" "vnet" {
@@ -40,7 +40,7 @@ resource "azurerm_subnet" "subnet" {
4040

4141
service_delegation {
4242
name = "Microsoft.Web/serverFarms"
43-
actions = ["Microsoft.Network/virtualNetworks/subnets/join/action", "Microsoft.Network/virtualNetworks/subnets/prepareNetworkPolicies/action"]
43+
actions = ["Microsoft.Network/virtualNetworks/subnets/action"]
4444
}
4545
}
4646

terraform/outputs.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
output "api_server_url" {
22
value = "https://${azurerm_linux_web_app.api_server_web_app.default_hostname}/"
3-
}
3+
}

0 commit comments

Comments
 (0)