timohirt / terraform-provider-hetznerdns

Terraform provider for Hetzner DNS
Mozilla Public License 2.0
106 stars 21 forks source link

Removing unneeded TTL's #26

Closed ThomasLandauer closed 3 years ago

ThomasLandauer commented 3 years ago

This PR is based on the assumption that adding a TTL for reach record doesn't add any benefit if it's the same as the zone's TTL. Is that true? If not, the PR is obsolete ;-)

timohirt commented 3 years ago

Hey @ThomasLandauer -

Your assumption is correct. The TTL of a record overrides the TTL that is specified for the zone and it is optional. Your change makes this clear.

Thanks for your contribution!