fix: Use computed local.security_group_name in the security group Name tag

[armanzeroeight]

Description

We should used the already computed local.security_group_name instead of var.name in security group rule name tag.

Motivation and Context

Its safer to use the local so if the user has decided to set a different name on SG than the EC2 stack.

security_group_name   = try(coalesce(var.security_group_name, var.name), "")

We are already using it for the SG name tag:

 tags = merge(
    var.tags,
    { "Name" = local.security_group_name },
    var.security_group_tags
  )

but not on the rules. This will harmonize that too.

Breaking Changes

No breaking changes

[antonbabenko]

This PR is included in version 6.1.2 🎉

[armanzeroeight]

Cool @antonbabenko and @bryantbiggs 🥇

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.