cloudposse / cloudposse/terraform-cloudflare-zone

Change records "proxied" default

Open
#47 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
HCL
Stars
35
Forks
28
PR merge metrics
No merged PRs in 30d

Description

### Describe the Feature

```tf
proxied = lookup(each.value, "proxied", false)
```
- https://github.com/cloudposse/terraform-cloudflare-zone/blob/3073cc39498d44ec44f023ffe1ec8d03d664ebf7/main.tf#L42

I would like to change proxied to `true` by default, as most of the records are proxied.

### Expected Behavior

Maybe introduce a new variable: `local.records_proxied_default`?

### Use Case

Reduce duplication.

Contributor guide

Open the contributing guide

Research direction

Start with main.tf around the records definition at line 42 and review how the proxied value is currently looked up. Make the default behavior true, potentially through the proposed local.records_proxied_default, and verify that record configurations no longer need to repeat proxied=true.

Written by the indexing model from the issue text.

Assessment

Tech stack
terraform
Domain
cloud, infrastructure
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.