Open
Conversation
当文件哈希与服务器上已有文件匹配时,file/update/hash API 会返回 finish: true,表示文件已存在无需上传。原代码忽略了这个返回值, 继续尝试上传到 OSS,导致 NoSuchBucket 错误。 现在检查 hash_result.finish 字段,如果为 true 则直接返回成功。 Fixes: lich0821#1, lich0821#2, lich0821#6 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
使用 file_path_obj.resolve() 将相对路径转换为绝对路径, 避免相对路径导致的异常行为 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
之前用 Path.cwd()/config,cd 后会找不到 cookie 导致要求重新登录 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题描述
当上传的文件哈希与服务器上已有文件匹配时(秒传场景),
file/update/hashAPI 会返回finish: true,表示文件已存在无需实际上传。但原代码忽略了这个返回值,继续尝试上传到 OSS,导致
NoSuchBucket错误。相关 Issues: #1, #2, #6
修复方案
在调用
_update_file_hash()后检查返回值中的finish字段:finish为True,直接返回上传成功(秒传完成)测试结果
代码变更