Add the custom domain name feature in aws_cognito_user_pool_domain resource

[adelamarre]

Fixes #5026

Changes proposed in this pull request:

• Add the certificate_arn parameter to let use a custom domain

Output from acceptance testing:

You need to set environment variable for this test :

• AWS_COGNITO_USER_POOL_DOMAIN_ROOT_DOMAIN
  This environment variable must be set to the fqdn of an ISSUED ACM certificate in us-east-1 to enable this test. The fqdn must have a A record in the zone.
• AWS_COGNITO_USER_POOL_DOMAIN_CERTIFICATE_ARN
  This environment variable must be set to the ARN of an ISSUED ACM certificate in us-east-1 to enable this test.

$ make testacc TESTARGS='-run=TestAccAWSCognitoUserPoolDomain_custom'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./... -v -run=TestAccAWSCognitoUserPoolDomain_custom -timeout 120m
?   	github.com/terraform-providers/terraform-provider-aws	[no test files]
=== RUN   TestAccAWSCognitoUserPoolDomain_custom
--- PASS: TestAccAWSCognitoUserPoolDomain_custom (2013.27s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	2013.299s

[bflad]

Hi @adelamarre 👋 Thanks for submitting this! Off to a great start -- I think just these two little things then it should be good to go. Please let us know if you have any questions or do not have time to implement the feedback. 👍

[bflad]

Thanks for this, @adelamarre! I was able to verify this using the environment variables with a public ACM certificate and domain. 🚀

--- PASS: TestAccAWSCognitoUserPoolDomain_basic (9.03s)
--- PASS: TestAccAWSCognitoUserPoolDomain_custom (1563.45s)

[bflad]

This has been released in version 1.42.0 of the AWS provider. 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!