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

Cannot specify type = drop for cloudflare_email_routing_rule #2448

Closed
limejuny opened this issue May 18, 2023 · 2 comments · Fixed by #2449
Closed

Cannot specify type = drop for cloudflare_email_routing_rule #2448

limejuny opened this issue May 18, 2023 · 2 comments · Fixed by #2449
Labels
kind/enhancement Categorizes issue or PR as related to improving an existing feature. workflow/pending-public-documentation Indicates an issue or PR requires changes to public documentation confirming suitability for use.
Milestone

Comments

@limejuny
Copy link
Contributor

Current Terraform and Cloudflare provider version

Terraform v1.4.2
on linux_arm64

  • provider registry.terraform.io/carlpett/sops v0.7.2
  • provider registry.terraform.io/cloudflare/cloudflare v4.5.0

Your version of Terraform is out of date! The latest version
is 1.4.6. You can update by downloading from https://www.terraform.io/downloads.html

Description

Email rule actions should be one of forward, worker, drop. (https://developers.cloudflare.com/email-routing/setup/email-routing-addresses/#email-rule-actions)
But there was no drop in latest provider.
I think there should also be drop option.

Use cases

I want to catch all email to one destination, exclude specific address.
So I create new cloudflare_email_routing_rule resource, but there was no drop options.
I would also like to manage address (which will be dropped) with terraform.

Potential Terraform configuration

...
resource "cloudflare_email_routing_rule" "spam" {
  zone_id = cloudflare_zone.this.id
  name    = "spam"
  enabled = true

  matcher {
    type  = "literal"
    field = "to"
    value = "spam@custom.domain"
  }

  action {
    type  = "drop"
    value = []
  }
}

References

#1937
#1947

@limejuny limejuny added kind/enhancement Categorizes issue or PR as related to improving an existing feature. needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels May 18, 2023
@github-actions
Copy link
Contributor

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 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.

@jacobbednarz jacobbednarz added workflow/pending-public-documentation Indicates an issue or PR requires changes to public documentation confirming suitability for use. and removed needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. labels May 18, 2023
@github-actions github-actions bot added this to the v4.20.0 milestone Nov 23, 2023
Copy link
Contributor

This functionality has been released in v4.20.0 of the Terraform Cloudflare Provider.

Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/enhancement Categorizes issue or PR as related to improving an existing feature. workflow/pending-public-documentation Indicates an issue or PR requires changes to public documentation confirming suitability for use.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants