fix(trace): URLCause 用层数上限终止,去掉不可比较的 error 比较

手写终止条件只认「长度 1 的自引用」,于是:
- 两节点环(a.Err=b / b.Err=a)⇒ 死循环,打挂当前请求的 goroutine;
- uerr.Err == err 在两侧动态类型相同且不可比较时 ⇒ panic
  (comparing uncomparable type),同样打挂当前请求。

改为唯一终止条件 maxURLCauseDepth=8:超限返回只含 Op 的替身,不返回当前层
(当前层是 *url.Error,Error() 里就是整条地址,不能为防死循环把地址漏回去)。
正常单层/双层路径逐字不变。用例:两节点环 1s 内返回、不可比较类型不 panic、
自引用仍退化成替身、超限路径无 URL 片段、上限内(8 层)仍剥到内层原因。
This commit is contained in:
AskaEth committed 2026-10-26 19:01:15 +08:00
1 parent 0ec9cbe32b
commit 92b0f89492
2 files changed
+168 -18

No files matched your search

+23 -6
View File
@@ -35,23 +35,40 @@ import (
// - **非 *url.Error 原样返回**:同一个值、不加包装,可安全当作「可能剥壳、也可能
// 什么都没做」的透传。
//
// 唯一的例外是 `*url.Error` 没有内层原因(Err 为 nil)或链自引用:那种错误**无法**
// 在不带地址的前提下原样返回,于是退化成只含操作名(Get/Post/parse)的替身 —— 宁可
// 少一条原因,也不回落到原串(回落就是泄漏)。net/http 的两处构造都填了 Err,因此
// 这条路在真实调用中到不了。
// 唯一的例外是 `*url.Error` 没有内层原因(Err 为 nil)或**层数超限**:那种错误
// **无法**在不带地址的前提下原样返回,于是退化成只含操作名(Get/Post/parse)的替身
// —— 宁可少一条原因,也不回落到原串(回落就是泄漏)。net/http 的两处构造都填了
// Err,且叠不过两层,因此这条路在真实调用中到不了。
//
// 终止条件**只靠层数上限**(maxURLCauseDepth),不靠「拿两个 error 比相等」:
// - 比相等挡不住**多节点环**(a.Err=b / b.Err=a),那种链会一直转到栈外的调用方
// 超时为止 —— 一个错误路径上的死循环等于打挂当前请求的 goroutine;
// - 接口值 `==` 在两侧动态类型相同且**不可比较**(含 slice/map 的错误实现)时
// **panic**(`comparing uncomparable type`),而 panic 同样打挂当前请求的
// goroutine。今天 13 个调用点传的都是 net/http 现场构造的可比较无环错误,因此
// 这条路不可达;但「今天不可达」会随时间衰减,而护栏的成本只是一次计数。
//
// 超限时的口径:**返回只含 Op 的替身,不返回当前层**。当前层仍是 `*url.Error`,
// 它的 Error() 里就是整条地址 —— 为防死循环而把地址带出去,正是本文件要修的病害
// 反过来复发。诊断面只丢「更内层的原因」,status/host 由调用方的文案补齐。
func URLCause(err error) error {
for {
for depth := 0; ; depth++ {
var uerr *url.Error
if !errors.As(err, &uerr) {
return err
}
if uerr.Err == nil || uerr.Err == err {
if uerr.Err == nil || depth >= maxURLCauseDepth {
return urlOpOnly(uerr.Op)
}
err = uerr.Err
}
}
// maxURLCauseDepth 是剥壳的层数上限,**唯一的终止条件**(见 URLCause)。取 8 的
// 理由:真实链最多两层(`client.Do` 包一次、调用方再 `%w` 一层),8 是它的四倍余量,
// 正常路径永远碰不到;同时小到环/超深链在微秒级就停住,不会拖住请求。
const maxURLCauseDepth = 8
// urlOpOnly 是「*url.Error 没有可用内层原因」时的替身:只留操作名,地址一个字节都不留。
func urlOpOnly(op string) error {
return errors.New("url " + op + " failed")