mirror/stackexchange-dnscontrol
1
0
Fork 0
mirror of https://github.com/StackExchange/dnscontrol.git synced 2024-05-11 05:55:12 +00:00
Code Issues Releases Activity

Gandi does not support TTLs > 30 days (#139)

Browse Source 
* Fix https://github.com/StackExchange/dnscontrol/issues/108

TODO(tlim): Add tests.

* fixup!
This commit is contained in:
Tom Limoncelli
2017-06-12 11:38:26 -04:00
committed by Craig Peterson
parent f23e67daf7
commit 2b93e6725f
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
providers/gandi/gandiProvider.go
View File

@ -8,6 +8,7 @@ import (
"github.com/StackExchange/dnscontrol/models" "github.com/StackExchange/dnscontrol/models"
"github.com/StackExchange/dnscontrol/providers" "github.com/StackExchange/dnscontrol/providers"
"github.com/StackExchange/dnscontrol/providers/diff" "github.com/StackExchange/dnscontrol/providers/diff"
"github.com/pkg/errors"
"strings" "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) log.Printf("WARNING: Gandi does not support ttls < 300. %s will not be set to %d.", rec.NameFQDN, rec.TTL)
rec.TTL = 300 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" { if rec.Type == "TXT" {
rec.Target = "\"" + rec.Target + "\"" // FIXME(tlim): Should do proper quoting. rec.Target = "\"" + rec.Target + "\"" // FIXME(tlim): Should do proper quoting.
} }