fix lb_target_group health check inconsistencies

[eguven]

• path defaults to "/" (which is what docs say) and validate function checks for "/" prefix (from AWS: "The destination path for health checks. This path must begin with a '/' character, and can be at most 1024 characters in length."
• timeout doc reflects the actual default (10) in code. AWS default is actually 5 but I did not want to change that and possibly break users' configuration.

When path is not provided the following error is produced:

Error: Error applying plan:

1 error(s) occurred:

aws_alb_target_group.some-tg-name: Error creating LB Target Group: InvalidParameter: 1 validation error(s) found.
- minimum field size of 1, CreateTargetGroupInput.HealthCheckPath.

[radeksimko]

Hi @eguven
thanks for the PR.

I'm ok with the docs update and validation improvement.

Do you mind removing the Default for path (i.e. leaving it as is)?

[radeksimko]

This was intentionally removed in order to bring support for network load balancers where path is irrelevant.

See 79467dc#diff-375aea487c27a6ada86edfd817ba2401

[eguven]

@radeksimko Thanks for the response, done. I hadn't considered network load balancers, sorry.

[bflad]

This has been released in terraform-provider-aws version 1.7.0. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!