fix

Author: wxiaoguang

routers/api/packages/container/container.go

@@ -312,8 +312,8 @@ func InitiateUploadBlob(ctx *context.Context) {
 	}
 
 	setResponseHeaders(ctx.Resp, &containerHeaders{
-		Location:   fmt.Sprintf("/v2/%s/%s/blobs/uploads/%s", ctx.Package.Owner.LowerName, image, upload.ID),
-		Range:      "0-0",
+		Location: fmt.Sprintf("/v2/%s/%s/blobs/uploads/%s", ctx.Package.Owner.LowerName, image, upload.ID),
+		// Range:      "0-0", // FIXME: why it is needed? Spec doesn't mention it, and it is not right: "0-0" means the first byte (len=1)
 		UploadUUID: upload.ID,
 		Status:     http.StatusAccepted,
 	})
@@ -334,7 +334,7 @@ func GetUploadBlob(ctx *context.Context) {
 	}
 
 	setResponseHeaders(ctx.Resp, &containerHeaders{
-		Range:      fmt.Sprintf("0-%d", upload.BytesReceived),
+		// Range:      fmt.Sprintf("0-%d", upload.BytesReceived), // FIXME: not right, the "range-end" should be "len-1"
 		UploadUUID: upload.ID,
 		Status:     http.StatusNoContent,
 	})
@@ -378,8 +378,8 @@ func UploadBlob(ctx *context.Context) {
 	}
 
 	setResponseHeaders(ctx.Resp, &containerHeaders{
-		Location:   fmt.Sprintf("/v2/%s/%s/blobs/uploads/%s", ctx.Package.Owner.LowerName, image, uploader.ID),
-		Range:      fmt.Sprintf("0-%d", uploader.Size()-1),
+		Location: fmt.Sprintf("/v2/%s/%s/blobs/uploads/%s", ctx.Package.Owner.LowerName, image, uploader.ID),
+		// Range:      fmt.Sprintf("0-%d", uploader.Size()-1), // FIXME: not right, when "uploader.Size()" is 0, it results in "0--1" which is not a valid range
 		UploadUUID: uploader.ID,
 		Status:     http.StatusAccepted,
 	})