-
Notifications
You must be signed in to change notification settings - Fork 632
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
Page Rules bugfix / enhancements #61
Conversation
…_override add resolve_override page_rule action
…_override add doc for resolve_override
…ader_override add host_header_override page_rule action
…ader_override missing bracket after merge conflict
…_override fix typo and go fmt
…ader_override go fmt
…cache_on_cookie Add page rule bypass cache on cookie
page_rules status is active or disabled not paused
add waf page rule
add doc for waf page rule keyword
@prdonahue @catsby @grubernaut please take a look. |
We are migrating our DNS provider from Route53 to Cloudflare. We are using terraform for that, but I faced some limitation related to S3 buckets once we have custom names for these buckets. Host_header_override is missing in the currect version of Cloudflare provider. We look forward to see this merged. |
We're also using the changes in this PR (along with some in #68) to start managing our page rules as code, and looking forward to seeing it merged. Thanks @SteveGoldthorpe-Work! |
@leandrocostam @simpsora I've put in a request to cloudflare support to make this higher priority for them. I encourage y'all to do the same so they have official data in their system of this customer demand. |
I've been in contact with Cloudflare's Product Manager for Terraform and they are fully aware of the issues and are currently working on getting development resources the relevant access. This all sounds encouraging. |
@SteveGoldthorpe-Work some of this has already been merged as part of https://github.com/terraform-providers/terraform-provider-cloudflare/pull/76 -- specifically The other two changes (addition of |
Opened a couple of new PRs: |
Fixes issues: