plugin/etcd: make the address of upstream optional (#2262)

Automatically submitted.

plugin/etcd/setup.go

@@ -92,9 +92,6 @@ func etcdParse(c *caddy.Controller) (*Etcd, bool, error) {
 					endpoints = args
 				case "upstream":
 					args := c.RemainingArgs()
-					if len(args) == 0 {
-						return nil, false, c.ArgErr()
-					}
 					u, err := upstream.New(args)
 					if err != nil {
 						return nil, false, err

plugin/etcd/setup_test.go

@@ -30,6 +30,20 @@ func TestSetupEtcd(t *testing.T) {
 	endpoint localhost:300
 }
 `, false, "skydns", []string{"localhost:300"}, "",
+		},
+		//test for upstream
+		{
+			`etcd {
+	endpoint localhost:300
+	upstream 8.8.8.8:53 8.8.4.4:53
+}`, false, "skydns", []string{"localhost:300"}, "",
+		},
+		//test for optional upstream address
+		{
+			`etcd {
+	endpoint localhost:300
+	upstream
+}`, false, "skydns", []string{"localhost:300"}, "",
 		},
 		// negative
 		{