Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs]: Incomplete valid values of security policy of resource aws_apigatewayv2_domain_name #37869

Open
nikpivkin opened this issue Jun 7, 2024 · 5 comments · May be fixed by #39479
Open

[Docs]: Incomplete valid values of security policy of resource aws_apigatewayv2_domain_name #37869

nikpivkin opened this issue Jun 7, 2024 · 5 comments · May be fixed by #39479
Labels
documentation Introduces or discusses updates to documentation. service/apigatewayv2 Issues and PRs that pertain to the apigatewayv2 service.

Comments

@nikpivkin
Copy link

Documentation Link

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/apigatewayv2_domain_name

Description

The documentation says that the valid values for the security policy (security_policy argument) are TLS_1_2. But TLS_1_0 is also valid. See references.

References

Would you like to implement a fix?

No

@nikpivkin nikpivkin added the documentation Introduces or discusses updates to documentation. label Jun 7, 2024
Copy link

github-actions bot commented Jun 7, 2024

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.

@terraform-aws-provider terraform-aws-provider bot added the needs-triage Waiting for first response or review from a maintainer. label Jun 7, 2024
@acwwat
Copy link
Contributor

acwwat commented Jun 7, 2024

The resource code allows only TLS_1_2 for security_policy and REGIONAL for endpoint_type seemingly for good reason. If I allow these values, applying the configuration ends up failing anyway with the following server-side validation errors:

Error: creating API Gateway v2 Domain Name (tf-acc-test-9084719555279808942.example.com): operation error ApiGatewayV2: CreateDomainName, https response error StatusCode: 400, RequestID: c57f749a-7e7e-4e43-bb77-537ac13d27c7, BadRequestException: TLS_1_0 is not supported for APIGatewayV2 domainName.
Error: creating API Gateway v2 Domain Name (tf-acc-test-4160857730709957401.example.com): operation error ApiGatewayV2: CreateDomainName, https response error StatusCode: 400, RequestID: 9fce6f3b-f32f-48be-a37e-aa2c422c2b76, BadRequestException: EDGE endpoint type is not supported for APIGatewayV2 domainName.

From usability standpoint it's better to leave the provider-side validation as-is.

@nikpivkin
Copy link
Author

@acwwat Is your API configured as private? Private APIs only support TLS 1.2.

@acwwat
Copy link
Contributor

acwwat commented Jun 7, 2024

@acwwat Is your API configured as private? Private APIs only support TLS 1.2.

I was validating the changes via acceptance tests, and looking at the test case the aws_apigatewayv2_domain_name is not associated with any API resources. But it is associated with a public domain (zone) and cert.

@justinretzolk justinretzolk added service/apigatewayv2 Issues and PRs that pertain to the apigatewayv2 service. and removed needs-triage Waiting for first response or review from a maintainer. labels Jun 11, 2024
@Jyots6914
Copy link
Contributor

I will raise PR to update the documentation.

@Jyots6914 Jyots6914 linked a pull request Sep 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Introduces or discusses updates to documentation. service/apigatewayv2 Issues and PRs that pertain to the apigatewayv2 service.
Projects
None yet
4 participants