From d5e3b9d8434f4a5fb28b3f9090df4629e7b1f696 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Patrick=20Sodre=CC=81?= <sodre@sodre.co>
Date: Thu, 10 Aug 2017 01:00:06 -0400
Subject: [PATCH] Refactor formatTxtRecords as encodeKVasRFC1464

  - Move the logic of rfc1035 out of the encoding function
  - Left basic version of encodingKV as 'k=v'
---
 agent/dns.go | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/agent/dns.go b/agent/dns.go
index 5259ae24a6..ce281de152 100644
--- a/agent/dns.go
+++ b/agent/dns.go
@@ -536,15 +536,9 @@ RPC:
 	}
 }
 
-// formatTxtRecords takes a kv-map and returns it as "[k=]v" for non-empty k not starting with rfc1035-
-func (d *DNSServer) formatTxtRecords(meta map[string]string) (txt []string) {
-	for k, v := range meta {
-		if strings.HasPrefix(k, "rfc1035-") {
-			txt = append(txt, v)
-		} else {
-			txt = append(txt, k+"="+v)
-		}
-	}
+// encodeKVasRFC1464 encodes a key-value pair according to RFC1464
+func (d *DNSServer) encodeKVasRFC1464(key, value string) (txt string) {
+	txt = key + "=" + value
 	return txt
 }
 
@@ -611,7 +605,11 @@ func (d *DNSServer) formatNodeRecord(node *structs.Node, addr, qName string, qTy
 	}
 
 	if node != nil && (qType == dns.TypeANY || qType == dns.TypeTXT) {
-		for _, txt := range d.formatTxtRecords(node.Meta) {
+		for key, value := range node.Meta {
+			txt := value
+			if !strings.HasPrefix(strings.ToLower(key), "rfc1035-") {
+				txt = d.encodeKVasRFC1464(key, value)
+			}
 			records = append(records, &dns.TXT{
 				Hdr: dns.RR_Header{
 					Name:   qName,