Skip to content

Commit

Permalink
Merge pull request #74 from Shenali-SJ/main
Browse files Browse the repository at this point in the history
[main] Update SSL profile configuration in Application Gateway
  • Loading branch information
ayeshajay authored May 16, 2024
2 parents 3d234b1 + 1a40356 commit 41a7a82
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 14 deletions.
13 changes: 5 additions & 8 deletions modules/azurerm/Application-Gateway/application_gateway.tf
Original file line number Diff line number Diff line change
Expand Up @@ -88,19 +88,15 @@ resource "azurerm_application_gateway" "app_gateway" {
}
}

ssl_policy {
policy_type = "Predefined"
policy_name = var.ssl_policy_name
}

dynamic "ssl_profile" {
for_each = var.ssl_profiles

content {
name = ssl_profile.value.name
ssl_policy {
policy_type = ssl_profile.value.ssl_policy.profile_policy_type
cipher_suites = ssl_profile.value.ssl_policy.profile_cipher_suites
min_protocol_version = ssl_profile.value.ssl_policy.min_protocol_version
policy_type = ssl_profile.value.ssl_policy.profile_policy_type
cipher_suites = ssl_profile.value.ssl_policy.profile_cipher_suites
}
}
}
Expand Down Expand Up @@ -271,7 +267,8 @@ resource "azurerm_application_gateway" "app_gateway" {
redirect_configuration,
tags,
trusted_root_certificate,
identity
identity,
trusted_client_certificate
]
}
}
7 changes: 1 addition & 6 deletions modules/azurerm/Application-Gateway/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -160,18 +160,13 @@ variable "waf_exclusion_settings" {
type = list(map(string))
}

variable "ssl_policy_name" {
default = "AppGwSslPolicy20170401S"
description = "Name of the SSLPolicy to use with Appgw"
type = string
}

variable "ssl_profiles" {
default = {}
description = "The SSL profile to be associate with a listener"
type = map(object({
name = string
ssl_policy = object({
min_protocol_version = string
profile_policy_type = string
profile_cipher_suites = list(string)
})
Expand Down

0 comments on commit 41a7a82

Please sign in to comment.