Merge pull request #152 from EmmEff/unauthorized-fix

Resolve issue with unauthenticated requests

Author: EmmEff

client/downloader.go

@@ -108,8 +108,11 @@ func (c *Client) ociDownloadBlobPart(ctx context.Context, creds credentials, u s
 	if err != nil {
 		return 0, err
 	}
-	if err := creds.ModifyRequest(req); err != nil {
-		return 0, err
+
+	if creds != nil {
+		if err := creds.ModifyRequest(req); err != nil {
+			return 0, err
+		}
 	}
 
 	req.Header.Add("Range", fmt.Sprintf("bytes=%d-%d", ps.start, ps.end))

client/oci.go

@@ -112,7 +112,9 @@ type bearerTokenCredentials struct {
 }
 
 func (c bearerTokenCredentials) ModifyRequest(r *http.Request, opts ...modifyRequestOption) error {
-	r.Header.Set("Authorization", fmt.Sprintf("Bearer %v", c.authToken))
+	if c.authToken != "" {
+		r.Header.Set("Authorization", fmt.Sprintf("Bearer %v", c.authToken))
+	}
 	return nil
 }
 

client/pull.go

@@ -236,8 +236,13 @@ func (c *Client) legacyDownloadImage(ctx context.Context, arch, name, tag string
 		return err
 	}
 
+	var creds credentials
+	if c.AuthToken != "" {
+		creds = bearerTokenCredentials{authToken: c.AuthToken}
+	}
+
 	// Use uri from Location header to download artifact
-	return c.multipartDownload(ctx, res.Header.Get("Location"), bearerTokenCredentials{authToken: c.AuthToken}, dst, img.Size, spec, pb)
+	return c.multipartDownload(ctx, res.Header.Get("Location"), creds, dst, img.Size, spec, pb)
 }
 
 func parseContentLengthHeader(val string) (int64, error) {