Avoid returning nil, nil when fetching a manifest by tag by introducing a new

error ErrManifestNotModified which can be checked by clients.

Signed-off-by: Richard Scothern <richard.scothern@gmail.com>

errors.go

@@ -1,15 +1,20 @@
 package distribution
 
 import (
+	"errors"
 	"fmt"
 	"strings"
 
 	"github.com/docker/distribution/digest"
 )
 
+// ErrManifestNotModified is returned when a conditional manifest GetByTag
+// returns nil due to the client indicating it has the latest version
+var ErrManifestNotModified = errors.New("manifest not modified")
+
 // ErrUnsupported is returned when an unimplemented or unsupported action is
 // performed
-var ErrUnsupported = fmt.Errorf("operation unsupported")
+var ErrUnsupported = errors.New("operation unsupported")
 
 // ErrRepositoryUnknown is returned if the named repository is not known by
 // the registry.

registry/client/repository.go

@@ -288,7 +288,7 @@ func (ms *manifests) GetByTag(tag string, options ...distribution.ManifestServic
 	}
 	defer resp.Body.Close()
 	if resp.StatusCode == http.StatusNotModified {
-		return nil, nil
+		return nil, distribution.ErrManifestNotModified
 	} else if SuccessStatus(resp.StatusCode) {
 		var sm schema1.SignedManifest
 		decoder := json.NewDecoder(resp.Body)

registry/client/repository_test.go

@@ -603,13 +603,10 @@ func TestManifestFetchWithEtag(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	m2, err := ms.GetByTag("latest", AddEtagToTag("latest", d1.String()))
+	_, err = ms.GetByTag("latest", AddEtagToTag("latest", d1.String()))
-	if err != nil {
+	if err != distribution.ErrManifestNotModified {
 		t.Fatal(err)
 	}
-	if m2 != nil {
-		t.Fatal("Expected empty manifest for matching etag")
-	}
 }
 
 func TestManifestDelete(t *testing.T) {

registry/proxy/proxymanifeststore.go

@@ -102,11 +102,11 @@ func (pms proxyManifestStore) GetByTag(tag string, options ...distribution.Manif
 fromremote:
 	var sm *schema1.SignedManifest
 	sm, err = pms.remoteManifests.GetByTag(tag, client.AddEtagToTag(tag, localDigest.String()))
-	if err != nil {
+	if err != nil && err != distribution.ErrManifestNotModified {
 		return nil, err
 	}
 
-	if sm == nil {
+	if err == distribution.ErrManifestNotModified {
 		context.GetLogger(pms.ctx).Debugf("Local manifest for %q is latest, dgst=%s", tag, localDigest.String())
 		return localManifest, nil
 	}