严重程度：中高影响范围：所有启用 LINE 通道的 OpenClaw 部署

修复前的实现：

const hashBuffer = Buffer.from(hash);const signatureBuffer = Buffer.from(signature);// Use constant-time comparison to prevent timing attacks.if (hashBuffer.length !== signatureBuffer.length) {  return false;}return crypto.timingSafeEqual(hashBuffer, signatureBuffer);

修复前后的控制流对比：左侧只要长度不匹配，就会直接走旁路；右侧将两个待比较的数据填充到相同长度，然后无条件执行完整的常量时间比较，从而消除长度不等时跳过受信比较原语的明显快路径。

差分计时实验：修复前组 A / B 的耗时均值相差约 84 微秒，远超网络抖动；修复后两组分布完全重合。

// Pad to equal length before constant-time comparison to prevent// leaking length information via early-return timing.const maxLen = Math.max(hashBuffer.length, signatureBuffer.length);const paddedHash = Buffer.alloc(maxLen);const paddedSig = Buffer.alloc(maxLen);hashBuffer.copy(paddedHash);signatureBuffer.copy(paddedSig);// Call timingSafeEqual unconditionally to ensure constant-time execution// regardless of length mismatch (avoids && short-circuit timing leak).const timingResult = crypto.timingSafeEqual(paddedHash, paddedSig);return hashBuffer.length === signatureBuffer.length && timingResult;

改动新增 11 行。配套测试覆盖三种场景：长度等且内容等、长度等内容不等、长度不等，并显式断言 timingSafeEqual 在所有分支都被实际调用，避免后续维护者出于风格考量重新引入长度快路径。

if (signature.length !== expected.length) {  return false;}let result = 0;for (let i = 0; i < signature.length; i++) {  result |= signature.charCodeAt(i) ^ expected.charCodeAt(i);}return result === 0;

严重程度：中影响范围：所有以腾讯云/阿里云/HuggingFace/Replicate 为模型后端的 OpenClaw 部署

• 腾讯云：AKID 前缀的 SecretId（TencentCloud API Secret ID）；

• 阿里云：LTAI 前缀的 AccessKey ID；

• HuggingFace：hf_ 前缀的 access token；

• Replicate：r8_ 前缀的 API token。

脱敏管线对照：默认管线遗漏的 4 类前缀（红色 chip）与修复后全量覆盖（统一收敛为 REDACTED）

修复初版比较直接：往默认脱敏正则集合追加上述四类前缀，模式统一为 \b<prefix>[A-Za-z0-9]{10,}\b。但合入后社区报告了一类过度脱敏：Akida、akidney 这样的英文单词在大小写不敏感匹配下会被误判成腾讯云 SecretId。这是脱敏规则的经典权衡：

严重程度：中影响范围：所有以网关模式（gateway.mode=local 或集群模式）运行的 OpenClaw 部署

前三项 PR 都能在静态评审里识别出来，前提是评审者具备相应的密码学或合规经验。

#58179 属于运行时状态泄漏：短周期测试通常看不出来，只有在节点反复接入、断开的场景里，RSS 才会随时间持续爬升，最终触发 OOM。

OpenClaw gateway 维护了一份按节点（node）粒度组织的 pending work 状态集，结构是两级 Map：

修复前的代码在两条路径上正确清理了内层条目（acknowledge 删除最后一个显式条目、drain 时过期条目被清空），但从未在内层为空时回收外层 stateByNodeId 中的对应键。

“内层为空”与“外层不存在”被错误地合并为同一种状态——GC 意义上合并了，数据结构语义上没合并。

正常长连接节点不会触发这个问题。但部署里一旦出现大量短命节点（临时连接、推一条任务、ack 完即断开），每个这样的节点都会在 stateByNodeId 里留下一个空壳条目：

左：修复前后 stateByNodeId 的内容对照；

右：进程 RSS 时序。

修复前 RSS 线性爬升直至 OOM，修复后贴近基线

启动 gateway 进程，模拟 N 万次一次性节点：每个节点接入、提交一条 pending work、立即 ack 后断开。观察进程 RSS：

RSS(t)≈RSS0+α⋅N(t)（修复前）RSS(t)≈RSS0（修复后，ack 完成回落至基线）

修复前 RSS 与累计经过的节点数 N(t) 严格线性相关；修复后 RSS 维持在基线附近，每轮 ack 完成即回落，与上图右半边的曲线吻合。

修复点放在会令内层变空的两条调用栈末尾：acknowledge 删除最后一个显式条目，以及 drain 时过期条目被清空（pruneExpired 路径）。其余分支（部分 ack、新增 item、状态更新等）即便接触到内层也不会令其变空，不必触发 prune，避免引入无效开销：

function pruneStateIfEmpty(  stateByNodeId: Map<string, NodeState>,  nodeId: string,  state: NodeState,): void {  if (state.itemsById.size === 0) {    stateByNodeId.delete(nodeId);  }}

判定流见下图：drain / ack 完成 → 询问 itemsById 是否为空 → 是则从外层移除 → NodeState 异步进入 GC。配套 17 行测试覆盖三种状态分支：内层非空（保留外层）、内层为空但节点仍活跃（保留外层）、内层为空且节点已断开（回收外层），并以 performance.memoryUsage 断言长跑场景下 RSS 收敛在基线附近。

ack/drain 完成后由 pruneStateIfEmpty 在同一调用栈对齐外层 Map 与 NodeState 的生命周期

• 任何短路求值（&&、||、提前 return、长度断言、参数早退）都会破坏常量时间属性。

• 即使写出“形式上常量时间”的手写循环，JIT 优化器也可能在语义等价的前提下把它编译成 early-exit。

• 评审中遇到密码学等值比较、签名校验、token 比对路径上的手写循环或长度快路径，应当默认替换为 timingSafeEqual 等受信原语。

• 召回率：覆盖所有实际出现在日志里的凭证前缀，遗漏即缺口。

• 误报率：避免把正常业务子串当凭证遮蔽，否则运维会绕过脱敏层去看原始流量，反而扩大攻击面。#58162 的初版把 Akida、akidney 当成腾讯云 SecretId 就是典型案例，后续 commit 改为大小写敏感才修掉。

• 可读性：过度遮蔽会让日志丧失排障价值。

这四项修复覆盖三类常见边界：签名校验、日志脱敏、运行时状态管理。对应到评审动作，就是检查签名比较是否存在时序差异、debug 日志是否覆盖真实凭证前缀、长跑进程是否残留空壳状态。

后续评审按三条检查即可：密码学防御原语必须无条件调用；多层嵌套结构必须在同一调用栈上对齐生命周期；脱敏规则必须按真实语料做正负样本回归。

腾讯团队在 OpenClaw 上游还覆盖网关协议、插件 SDK、多通道适配等其他主线，本文只取其中安全与稳定性方向的一个子集。

本文涉及的 OpenClaw 上游 PR