Merge pull request #70 from njhale/fix/local-gc

Elide "local" records from purge set
acorn-io · May 24, 2023 · d188d56 · d188d56
2 parents 17a4686 + 0b97ed3
commit d188d56
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/pkg/backend/purger.go b/pkg/backend/purger.go
@@ -71,6 +71,14 @@ func (b *backend) purge() {
 		return
 	}
 
+	// Elide records for "local" FQDNs, these won't exist in the database but shouldn't be deleted.
+	localSuffix := ".local." + b.baseDomain
+	for pair := range recordsToDelete {
+		if strings.HasSuffix(pair.FQDN, localSuffix) {
+			delete(recordsToDelete, pair)
+		}
+	}
+
 	if len(recordsToDelete) == 0 {
 		logrus.Infof("Records purged from Route53: 0")
 		return