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

The validator of alicloud_cdn_domain_new.sources.port is incorrect #6891

Open
ElmerZhang opened this issue Jan 23, 2024 · 0 comments
Open

The validator of alicloud_cdn_domain_new.sources.port is incorrect #6891

ElmerZhang opened this issue Jan 23, 2024 · 0 comments

Comments

@ElmerZhang
Copy link

ElmerZhang commented Jan 23, 2024

The validator of alicloud_cdn_domain_new.sources.port is incorrect. According to the official website, the port can be any port from 1 to 65535, but the validator of it only allows for 80 and 443. I have created a pull request #6885 to fix it.

image

Terraform Version

Terraform v1.5.0
on darwin_amd64
+ provider registry.terraform.io/aliyun/alicloud v1.214.0

Affected Resource(s)

  • alicloud_cdn_domain_new

Terraform Configuration Files

resource "alicloud_cdn_domain_new" "somedomain" {
 scope       = "global"
 domain_name = "somedomain.fydeos.com"
 cdn_type    = "web"

 sources {
   type     = "ipaddr"
   content  = "12.23.34.45"
   priority = 20
   port     = 8880
   weight   = 10
 }
}

Panic Output

│ Error: expected sources.0.port to be one of [80 443], got 8880 
│ [NOTE] set env variable TF_SKIP_RESOURCE_SCHEMA_VALIDATION to true can skip the error and get a warning
│ 
│   with alicloud_cdn_domain_new.somedomain,
│   on cdn.tf line 162, in resource "alicloud_cdn_domain_new" "somedomain":
│  162: resource "alicloud_cdn_domain_new" "somedomain" {

References

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant