Merge pull request #22 from zoitech/fix/https_with_offloading

added prefix to security group lb_name
zoitech · Jul 28, 2020 · 9db1cc6 · 9db1cc6
2 parents afb9522 + 3ddbb0c
commit 9db1cc6
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## 1.0.4
+
+ENHANCEMENTS:
+
+* The security group name for "lb_group" now includes the prefix
+
 ## 1.0.3
 
 BUG FIX:

diff --git a/README.md b/README.md
@@ -166,7 +166,7 @@ The following outputs are available:
 
 ```hcl
 module "alb" {
-  source = "git::https://github.com/zoitech/terraform-aws-alb.git?ref=v1.0.3"
+  source = "git::https://github.com/zoitech/terraform-aws-alb.git?ref=v1.0.4"
 ```
 
 ## Authors

diff --git a/security_groups.tf b/security_groups.tf
@@ -1,7 +1,7 @@
 # Security group "group" for loadbalancer with no rules. 
 #To be used in other security group rules to specify the loadblaancer as the source.
 resource "aws_security_group" "lb_group" {
-  name        = "Group-ALB-${var.lb_name}"
+  name        = "Group-ALB-${var.prefix}-${var.lb_name}"
   description = "Attach security group with loadbalancer name to loadbalancer with no rules"
   vpc_id      = var.vpc_id
 }