Skip to content

Commit

Permalink
Merge pull request #1 from dasmeta/DMVP-1338-multidomain
Browse files Browse the repository at this point in the history
DMVP-1338: Modified domain variable to support lists
  • Loading branch information
VahagnMian authored Oct 24, 2022
2 parents 0f2857c + 281f5ca commit 0bc43cf
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions variables.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@

variable "domain" {
type = string
type = list(string)
description = "domain full name"
}

Expand All @@ -15,4 +14,3 @@ variable "waf" {
default = null
description = "waf configs"
}

2 changes: 1 addition & 1 deletion waf.tf
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ module "waf" {

providers = {
// TODO: for cloudfront distribution the waf gets created in virginia, but for alb the specific region should be used,
// needs to decide how to accomplish this
// needs to decide how to accomplish this
aws = aws.virginia
}
}

0 comments on commit 0bc43cf

Please sign in to comment.