fix(139): handle upload file conflicts (#7692)

drivers/139/driver.go

@@ -552,7 +552,7 @@ func (d *Yun139) Put(ctx context.Context, dstDir model.Obj, stream model.FileStr
 			firstPartInfos = firstPartInfos[:100]
 		}
 
-		// 获取上传信息和前100个分片的上传地址
+		// 创建任务，获取上传信息和前100个分片的上传地址
 		data := base.Json{
 			"contentHash":          fullHash,
 			"contentHashAlgorithm": "SHA256",
@@ -572,10 +572,17 @@ func (d *Yun139) Put(ctx context.Context, dstDir model.Obj, stream model.FileStr
 			return err
 		}
 
-		if resp.Data.Exist || resp.Data.RapidUpload {
+		// 判断文件是否已存在
+		// resp.Data.Exist: true 已存在同名文件且校验相同，云端不会重复增加文件，无需手动处理冲突
+		if resp.Data.Exist {
 			return nil
 		}
 
+		// 判断文件是否支持快传
+		// resp.Data.RapidUpload: true 支持快传，但此处直接检测是否返回分片的上传地址
+		// 快传的情况下同样需要手动处理冲突
+		if resp.Data.PartInfos != nil {
+			// 读取前100个分片的上传地址
 			uploadPartInfos := resp.Data.PartInfos
 
 			// 获取后续分片的上传地址
@@ -649,10 +656,72 @@ func (d *Yun139) Put(ctx context.Context, dstDir model.Obj, stream model.FileStr
 			if err != nil {
 				return err
 			}
+		}
+
+		// 处理冲突
+		if resp.Data.FileName != stream.GetName() {
+			log.Debugf("[139] conflict detected: %s != %s", resp.Data.FileName, stream.GetName())
+			// 给服务器一定时间处理数据，避免无法刷新文件列表
+			time.Sleep(time.Millisecond * 500)
+			// 刷新并获取文件列表
+			files, err := d.List(ctx, dstDir, model.ListArgs{Refresh: true})
+			if err != nil {
+				return err
+			}
+			// 删除旧文件
+			for _, file := range files {
+				if file.GetName() == stream.GetName() {
+					log.Debugf("[139] conflict: removing old: %s", file.GetName())
+					// 删除前重命名旧文件，避免仍旧冲突
+					err = d.Rename(ctx, file, stream.GetName()+random.String(4))
+					if err != nil {
+						return err
+					}
+					err = d.Remove(ctx, file)
+					if err != nil {
+						return err
+					}
+					break
+				}
+			}
+			// 重命名新文件
+			for _, file := range files {
+				if file.GetName() == resp.Data.FileName {
+					log.Debugf("[139] conflict: renaming new: %s => %s", file.GetName(), stream.GetName())
+					err = d.Rename(ctx, file, stream.GetName())
+					if err != nil {
+						return err
+					}
+					break
+				}
+			}
+		}
 		return nil
 	case MetaPersonal:
 		fallthrough
 	case MetaFamily:
+		// 处理冲突
+		// 获取文件列表
+		files, err := d.List(ctx, dstDir, model.ListArgs{})
+		if err != nil {
+			return err
+		}
+		// 删除旧文件
+		for _, file := range files {
+			if file.GetName() == stream.GetName() {
+				log.Debugf("[139] conflict: removing old: %s", file.GetName())
+				// 删除前重命名旧文件，避免仍旧冲突
+				err = d.Rename(ctx, file, stream.GetName()+random.String(4))
+				if err != nil {
+					return err
+				}
+				err = d.Remove(ctx, file)
+				if err != nil {
+					return err
+				}
+				break
+			}
+		}
 		data := base.Json{
 			"manualRename": 2,
 			"operation":    0,
@@ -688,7 +757,7 @@ func (d *Yun139) Put(ctx context.Context, dstDir model.Obj, stream model.FileStr
 			pathname = "/orchestration/familyCloud-rebuild/content/v1.0/getFileUploadURL"
 		}
 		var resp UploadResp
-		_, err := d.post(pathname, data, &resp)
+		_, err = d.post(pathname, data, &resp)
 		if err != nil {
 			return err
 		}

drivers/139/types.go

@@ -261,6 +261,7 @@ type PersonalUploadResp struct {
 	BaseResp
 	Data struct {
 		FileId      string             `json:"fileId"`
+		FileName    string             `json:"fileName"`
 		PartInfos   []PersonalPartInfo `json:"partInfos"`
 		Exist       bool               `json:"exist"`
 		RapidUpload bool               `json:"rapidUpload"`