Fix reading configuration for multiple custom domains

Without this patch, the different domains end up with pointers to the same
domainsConfig struct, as go re-uses the same region of memory on each iteration
of a for loop.

domains.go

@@ -44,8 +44,9 @@ func (d domains) readProjectConfig(rootDomain, group, project string) (err error
 	}
 	for _, domainConfig := range config.Domains {
+		config := domainConfig // domainConfig is reused for each loop iteration
 		if domainConfig.Valid(rootDomain) {
-			d.addDomain(rootDomain, group, project, &domainConfig)
+			d.addDomain(rootDomain, group, project, &config)
 		}
 	}
 	return

domains_test.go

@@ -40,6 +40,13 @@ func TestReadProjects(t *testing.T) {
 	for _, actual := range domains {
 		assert.Contains(t, expectedDomains, actual)
 	}
+
+	// Check that multiple domains in the same project are recorded faithfully
+	exp1 := &domainConfig{Domain: "test.domain.com"}
+	assert.Equal(t, exp1, d["test.domain.com"].Config)
+
+	exp2 := &domainConfig{Domain: "other.domain.com", Certificate: "test", Key: "key"}
+	assert.Equal(t, exp2, d["other.domain.com"].Config)
 }
 func writeRandomTimestamp() {

shared/pages/group/group.test.io/config.json

@@ -9,7 +9,7 @@
     {
       "Domain": "other.domain.com",
       "Certificate": "test",
-      "Certificate": "key"
+      "Key": "key"
     }
   ]
 }