vultr: Modernize record conversion - #4583
Conversation
|
I've been a little busy recently, but I'll review this on Saturday. |
|
Sounds good! I've added some TODO comments with suggestions. Please try them one at a time and let me know which works best. Thanks! |
|
Integration tests as of 7d60892: DetailsWill look at the TODOs and failures now. |
|
To work around the The
The TXT value is always returned surrounded with double-quotes, and with backslashes appearing literally within them. |
I will PR these changes. |
|
Thinking about the TTL issue a bit more:
|
|
Is dnscontrol/providers/gcloud/gcloudProvider.go Line 369 in 7d65f82 There does seem to be dnscontrol/pkg/normalize/validate.go Line 698 in 7d65f82 The gandiv5, huaweiclod, and gcore providers seem to just warn and use the smallest TTL, as implied by the comment in The gcloud handling seems to differ from the others I spot-checked. |
* vultr: Resolve TODOs * vultr: Properly handle TTL changes * vultr: Fix null MX handling * vultr: Update audit * vultr: Also reject CAA records with an empty target * vultr: Simplify GetZoneRecords * vultr: Implement ListZones * vultr: Drop unused field * vultr: Use github.com/vultr/govultr/v3 * vultr: Implement AutoDNSSEC * vultr: Verified concurrency
Good catch! That comment is wrong. I'll correct it. |
|
Great work! I'm merging this. Please look for v5.0.0-rc3 and test again and/or test Thanks! |
Dear @pgaskin. I could use your help. I'm doing upgrades to the code base and I have no way to test your provider. Please run the integration test and let me know the result. How to run integration tests is here: https://docs.dnscontrol.org/developer-info/integration-tests
Summary
RecordConfigconstruction and target setters with record factoriesTesting
../../bin/is_modern.sh(Steps 1-7 clean)go test ./providers/vultrbin/generate-all.shgo test ./...