Repository navigation
fix(inlineimg): 缩图临时文件加随机后缀，避免并发会话撞名 - #166
Merged
Merged
Conversation
临时名原来只有 pid + 毫秒，同毫秒开跑的两个会话会写/读/删同一个文件：一个会话的图 可能被另一个删掉或读到半截，静默丢图甚至串图。加 4 字节随机后缀。 本机 inlineimg 7 项全过。 Co-Authored-By: Claude Code <noreply@anthropic.com>
PR 安全初筛 / Safety pre-check改动 1 个文件；高 0 项、中 0 项。 没发现需要特别留意的地方。自动初筛只能挡明显的，合并前仍需人工看过。 |
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.
概要
临时名原来只有 pid + 毫秒，同毫秒开跑的两个会话会写/读/删同一个文件：一个会话的图可能被另一个删掉或读到半截，静默丢图甚至串图。
改动
测试
node scripts/test.mjs inlineimg7 项全过。定向测试
测试验证（本机 Android）
12 个修复合入集成分支后，全量网关测试 768 tests / 767 pass / 0 fail / 1 skip（1.1GB 大文件用例按设计跳过）；Java 测试（Ed25519/PrivRules/KeyAttest）、前端拼装（27 段/15052 行）、i18n 生成与扫描、全部 JS/shell/python 语法检查均通过。
🤖 Generated with Claude Code