Fix trailing comma in data source conf DNS

Fixes #2. Any config data source with at least one DNS value set was affected.
OJFord · Mar 31, 2021 · 735daa0 · 735daa0
1 parent 58ef3d8
commit 735daa0
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/docs/data-sources/config_document.md b/docs/data-sources/config_document.md
@@ -39,6 +39,10 @@ data "wireguard_config_document" "peer1" {
     ]
   }
 }
+
+output "peer1" {
+  value = data.wireguard_config_document.peer1.conf
+}
 ```
 
 <!-- schema generated by tfplugindocs -->

diff --git a/examples/data-sources/wireguard_config_document/data-source.tf b/examples/data-sources/wireguard_config_document/data-source.tf
@@ -24,3 +24,7 @@ data "wireguard_config_document" "peer1" {
     ]
   }
 }
+
+output "peer1" {
+  value = data.wireguard_config_document.peer1.conf
+}
diff --git a/provider/data_source_wireguard_config_document.go b/provider/data_source_wireguard_config_document.go
@@ -160,7 +160,7 @@ Address = {{ . }}
 {{- end }}
 
 {{- if .DNS }}
-DNS = {{ range .DNS }}{{ . }},{{ end }}
+DNS = {{ range $i, $d := .DNS }}{{ if $i }},{{ end }}{{ $d }}{{ end }}
 {{- end }}
 
 {{- if .MTU }}
-Original file line number
+Diff line change
@@ Expand Up / @@ -39,6 +39,10 @@ data "wireguard_config_document" "peer1" { @@
         ]
       }
     }
+    output "peer1" {
+      value = data.wireguard_config_document.peer1.conf
+    }
     ```
     <!-- schema generated by tfplugindocs -->
@@ Expand Down @@