Move ingress/egress to aws_security_group_rule in rds

When I make an RDS instance, it kept saying it needed to change stuff.
This is also best practice from what I can tell (if users of the module
need to add rules to the SG also)

Author: thattommyhall

rds/main.tf

@@ -87,32 +87,38 @@ variable "subnet_ids" {
   type        = "list"
 }
 
-resource "aws_security_group" "main" {
-  name        = "${var.name}-rds"
-  description = "Allows traffic to RDS from other security groups"
-  vpc_id      = "${var.vpc_id}"
+resource "aws_security_group_rule" "main-ingress-cidrs" {
+  type = "ingress"
+  cidr_blocks = ["${var.ingress_allow_cidr_blocks}"]
+  from_port       = "${var.port}"
+  to_port         = "${var.port}"
+  protocol        = "TCP"
+}
 
-  ingress {
-    from_port       = "${var.port}"
-    to_port         = "${var.port}"
-    protocol        = "TCP"
-    security_groups = ["${var.ingress_allow_security_groups}"]
-  }
+resource "aws_security_group_rule" "main-ingress-sgs" {
+  count = "${length(var.ingress_allow_security_groups)}"
+  source_security_group_id = "${element(var.ingress_allow_security_groups, count.index)}"
+  type = "ingress"
+  security_group_id = "${aws_security_group.main.id}"
+  from_port       = "${var.port}"
+  to_port         = "${var.port}"
+  protocol        = "TCP"
+}
 
-  ingress {
-    from_port   = "${var.port}"
-    to_port     = "${var.port}"
-    protocol    = "TCP"
-    cidr_blocks = ["${var.ingress_allow_cidr_blocks}"]
-  }
+resource "aws_security_group_rule" "main-egress-all" {
+  type        = "egress"
+  security_group_id = "${aws_security_group.main.id}"
+  from_port   = 0
+  to_port     = 0
+  protocol    = -1
+  cidr_blocks = ["0.0.0.0/0"]
+}
 
-  egress {
-    from_port   = 0
-    to_port     = 0
-    protocol    = -1
-    cidr_blocks = ["0.0.0.0/0"]
-  }
 
+resource "aws_security_group" "main" {
+  name        = "${var.name}-rds"
+  description = "Allows traffic to RDS from other security groups"
+  vpc_id      = "${var.vpc_id}"
   tags {
     Name = "RDS (${var.name})"
   }