From 2b93e6725fcd4ffb0e37c44e4499880d444a14ad Mon Sep 17 00:00:00 2001 From: Tom Limoncelli Date: Mon, 12 Jun 2017 11:38:26 -0400 Subject: [PATCH] Gandi does not support TTLs > 30 days (#139) * Fix https://github.com/StackExchange/dnscontrol/issues/108 TODO(tlim): Add tests. * fixup! --- providers/gandi/gandiProvider.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/providers/gandi/gandiProvider.go b/providers/gandi/gandiProvider.go index 27dcaffca..6f9eb9eb1 100644 --- a/providers/gandi/gandiProvider.go +++ b/providers/gandi/gandiProvider.go @@ -8,6 +8,7 @@ import ( "github.com/StackExchange/dnscontrol/models" "github.com/StackExchange/dnscontrol/providers" "github.com/StackExchange/dnscontrol/providers/diff" + "github.com/pkg/errors" "strings" @@ -77,6 +78,9 @@ func (c *GandiApi) GetDomainCorrections(dc *models.DomainConfig) ([]*models.Corr log.Printf("WARNING: Gandi does not support ttls < 300. %s will not be set to %d.", rec.NameFQDN, rec.TTL) rec.TTL = 300 } + if rec.TTL > 2592000 { + return nil, errors.Errorf("ERROR: Gandi does not support TTLs > 30 days (TTL=%d)", rec.TTL) + } if rec.Type == "TXT" { rec.Target = "\"" + rec.Target + "\"" // FIXME(tlim): Should do proper quoting. }