Merge pull request containerd#3484 from apostasie/fix-login-regression

Fix login persistence regression against Hub

Author: AkihiroSuda

pkg/imgutil/dockerconfigresolver/credentialsstore.go

@@ -83,7 +83,7 @@ func (cs *CredentialsStore) Store(registryURL *RegistryURL, credentials *Credent
 	if registryURL.Namespace != nil {
 		credentials.ServerAddress = fmt.Sprintf("%s%s?%s", registryURL.Host, registryURL.Path, registryURL.RawQuery)
 	} else {
-		credentials.ServerAddress = registryURL.Host
+		credentials.ServerAddress = registryURL.CanonicalIdentifier()
 	}
 
 	// XXX future namespaced url likely require special handling here

pkg/imgutil/dockerconfigresolver/credentialsstore_test.go

@@ -393,3 +393,5 @@ func TestWorkingCredentialsStore(t *testing.T) {
 
 	})
 }
+
+// TODO: add more tests that write credentials (specifically to hub locations) to verify they use the canonical id properly

pkg/imgutil/dockerconfigresolver/registryurl_test.go

@@ -38,6 +38,11 @@ func TestURLParsingAndID(t *testing.T) {
 			address: "whatever://",
 			error:   ErrUnsupportedScheme,
 		},
+		{
+			address:    "",
+			identifier: "https://index.docker.io/v1/",
+			allIDs:     []string{"https://index.docker.io/v1/"},
+		},
 		{
 			address:    "https://index.docker.io/v1/",
 			identifier: "https://index.docker.io/v1/",