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

Merge branch 'master' into tlim_corrector

Browse Source
This commit is contained in:
Tom Limoncelli
2023-03-09 05:18:52 -08:00
parent e10f169056 ad4b8daa13
commit ececcf5cee
3 changed files with 15 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
commands/previewPush.go
View File

@ -497,12 +497,14 @@ func printOrRunCorrections(domain string, provider string, corrections []*models
if interactive && !out.PromptToRun() {
continue
}
if correction.F != nil {
err = correction.F()
out.EndCorrection(err)
if err != nil {
anyErrors = true
}
}
}
notifier.Notify(domain, provider, correction.Msg, err, !push)
}
return anyErrors

13
integrationTest/integration_test.go
View File

@ -844,19 +844,18 @@ func makeTests(t *testing.T) []*TestGroup {
// of record. When the provider fixes the bug or changes behavior,
// update the AuditRecords().
tc("TXT with 0-octel string", txt("foo1", "")),
// NB(tlim) 2023-03-07: Removing this test. Nobody does this.
//tc("TXT with 0-octel string", txt("foo1", "")),
// https://github.com/StackExchange/dnscontrol/issues/598
// RFC1035 permits this, but rarely do provider support it.
//clear(),
tc("a 255-byte TXT", txt("foo255", strings.Repeat("C", 255))),
//tc("a 255-byte TXT", txt("foo255", strings.Repeat("C", 255))),
//clear(),
tc("a 256-byte TXT", txt("foo256", strings.Repeat("D", 256))),
//tc("a 256-byte TXT", txt("foo256", strings.Repeat("D", 256))),
//clear(),
tc("a 512-byte TXT", txt("foo512", strings.Repeat("C", 512))),
//tc("a 512-byte TXT", txt("foo512", strings.Repeat("C", 512))),
//clear(),
tc("a 513-byte TXT", txt("foo513", strings.Repeat("D", 513))),
//tc("a 513-byte TXT", txt("foo513", strings.Repeat("D", 513))),
//clear(),
tc("TXT with 1 single-quote", txt("foosq", "quo'te")),

7
providers/digitalocean/digitaloceanProvider.go
View File

@ -282,12 +282,11 @@ func toRc(domain string, r *godo.DomainRecord) *models.RecordConfig {
CaaFlag: uint8(r.Flags),
}
t.SetLabelFromFQDN(name, domain)
t.SetTarget(target)
switch rtype := r.Type; rtype {
case "TXT":
t.SetTargetTXTfromRFC1035Quoted(target)
t.SetTargetTXT(target)
default:
// nothing additional required
t.SetTarget(target)
}
return t
}
@ -309,7 +308,7 @@ func toReq(dc *models.DomainConfig, rc *models.RecordConfig) *godo.DomainRecordE
priority = int(rc.SrvPriority)
case "TXT":
// TXT records are the one place where DO combines many items into one field.
target = rc.GetTargetField()
target = rc.GetTargetTXTJoined()
default:
// no action required
}