[Docs]: It was not mentioned that cidr_blocks and security_groups are ORed

[EugenKon]

Documentation Link

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/security_group#ingress

Description

Terraform Version

NA

Affected Pages

What is the docs issue?

It does not mention, that separate rules are created for security_groups and cidr_blocks

  ingress {
    from_port   = 22
    to_port     = 22
    protocol    = "tcp"
    security_groups = "${var.client_vpn_sg}"
    cidr_blocks = local.allowlist_ip
  }

I suppose by mistake that inbound traffic should be from cidr_blocks AND from security_groups. But they are processed separately.

Proposal

Doc should clarify this.

References

https://github.com/hashicorp/terraform/issues/35398

References

No response

Would you like to implement a fix?

None

[github-actions[bot]]

Community Note

Voting for Prioritization

• Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
• Please see our prioritization guide for information on how we prioritize.
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

• If you are interested in working on this issue, please leave a comment.
• If this would be your first contribution, please review the contribution guide.