From 01dfa0fcb9af1d0c2b08e1008d8efcb95cbf5369 Mon Sep 17 00:00:00 2001 From: Stephen J Day Date: Thu, 15 Dec 2016 15:07:42 -0800 Subject: [PATCH] digest: use digest.Parse over ParseDigest Signed-off-by: Stephen J Day --- reference/reference.go | 2 +- registry/client/repository.go | 6 +++--- registry/handlers/blobupload.go | 4 ++-- registry/handlers/context.go | 2 +- registry/handlers/images.go | 2 +- registry/storage/blobstore.go | 2 +- registry/storage/vacuum.go | 2 +- 7 files changed, 10 insertions(+), 10 deletions(-) diff --git a/reference/reference.go b/reference/reference.go index 02786628e..29b949aad 100644 --- a/reference/reference.go +++ b/reference/reference.go @@ -170,7 +170,7 @@ func Parse(s string) (Reference, error) { } if matches[3] != "" { var err error - ref.digest, err = digest.ParseDigest(matches[3]) + ref.digest, err = digest.Parse(matches[3]) if err != nil { return nil, err } diff --git a/registry/client/repository.go b/registry/client/repository.go index 1ebd0b183..55dcf4392 100644 --- a/registry/client/repository.go +++ b/registry/client/repository.go @@ -268,7 +268,7 @@ func descriptorFromResponse(response *http.Response) (distribution.Descriptor, e return desc, nil } - dgst, err := digest.ParseDigest(digestHeader) + dgst, err := digest.Parse(digestHeader) if err != nil { return distribution.Descriptor{}, err } @@ -475,7 +475,7 @@ func (ms *manifests) Get(ctx context.Context, dgst digest.Digest, options ...dis return nil, distribution.ErrManifestNotModified } else if SuccessStatus(resp.StatusCode) { if contentDgst != nil { - dgst, err := digest.ParseDigest(resp.Header.Get("Docker-Content-Digest")) + dgst, err := digest.Parse(resp.Header.Get("Docker-Content-Digest")) if err == nil { *contentDgst = dgst } @@ -553,7 +553,7 @@ func (ms *manifests) Put(ctx context.Context, m distribution.Manifest, options . if SuccessStatus(resp.StatusCode) { dgstHeader := resp.Header.Get("Docker-Content-Digest") - dgst, err := digest.ParseDigest(dgstHeader) + dgst, err := digest.Parse(dgstHeader) if err != nil { return "", err } diff --git a/registry/handlers/blobupload.go b/registry/handlers/blobupload.go index 3afb47398..09a86fc5c 100644 --- a/registry/handlers/blobupload.go +++ b/registry/handlers/blobupload.go @@ -211,7 +211,7 @@ func (buh *blobUploadHandler) PutBlobUploadComplete(w http.ResponseWriter, r *ht return } - dgst, err := digest.ParseDigest(dgstStr) + dgst, err := digest.Parse(dgstStr) if err != nil { // no digest? return error, but allow retry. buh.Errors = append(buh.Errors, v2.ErrorCodeDigestInvalid.WithDetail("digest parsing failed")) @@ -329,7 +329,7 @@ func (buh *blobUploadHandler) blobUploadResponse(w http.ResponseWriter, r *http. // successful, the blob is linked into the blob store and 201 Created is // returned with the canonical url of the blob. func (buh *blobUploadHandler) createBlobMountOption(fromRepo, mountDigest string) (distribution.BlobCreateOption, error) { - dgst, err := digest.ParseDigest(mountDigest) + dgst, err := digest.Parse(mountDigest) if err != nil { return nil, err } diff --git a/registry/handlers/context.go b/registry/handlers/context.go index 552db2df6..fb883f60d 100644 --- a/registry/handlers/context.go +++ b/registry/handlers/context.go @@ -62,7 +62,7 @@ func getDigest(ctx context.Context) (dgst digest.Digest, err error) { return "", errDigestNotAvailable } - d, err := digest.ParseDigest(dgstStr) + d, err := digest.Parse(dgstStr) if err != nil { ctxu.GetLogger(ctx).Errorf("error parsing digest=%q: %v", dgstStr, err) return "", err diff --git a/registry/handlers/images.go b/registry/handlers/images.go index c51bbddb9..5ed1e60c1 100644 --- a/registry/handlers/images.go +++ b/registry/handlers/images.go @@ -33,7 +33,7 @@ func imageManifestDispatcher(ctx *Context, r *http.Request) http.Handler { Context: ctx, } reference := getReference(ctx) - dgst, err := digest.ParseDigest(reference) + dgst, err := digest.Parse(reference) if err != nil { // We just have a tag imageManifestHandler.Tag = reference diff --git a/registry/storage/blobstore.go b/registry/storage/blobstore.go index 4274cc9e8..6ec2e66a0 100644 --- a/registry/storage/blobstore.go +++ b/registry/storage/blobstore.go @@ -145,7 +145,7 @@ func (bs *blobStore) readlink(ctx context.Context, path string) (digest.Digest, return "", err } - linked, err := digest.ParseDigest(string(content)) + linked, err := digest.Parse(string(content)) if err != nil { return "", err } diff --git a/registry/storage/vacuum.go b/registry/storage/vacuum.go index 3bdfebf27..7302f733f 100644 --- a/registry/storage/vacuum.go +++ b/registry/storage/vacuum.go @@ -29,7 +29,7 @@ type Vacuum struct { // RemoveBlob removes a blob from the filesystem func (v Vacuum) RemoveBlob(dgst string) error { - d, err := digest.ParseDigest(dgst) + d, err := digest.Parse(dgst) if err != nil { return err }