AskaEth
4c187ac070
五处逐字同形的三行（mediafetch.sanitizeErr / httpclient.transportCause / vision.fetchCause / telegram.callCause / weixin_ilink.urlFreeCause）收敛为 trace 包里的唯一实现，13 个调用点改为直接调用；5 份 body 只剩 1 份，5 个 未导出符号与其文件内注释整份删除（不留薄封装）。 行为等价：只剥 *url.Error、不新增包装，内层哨兵链（context 取消/超时、 *net.DNSError 等）原样透传，因此重试/降级/熔断的分类判据不变。两处刻意的 收紧： - 逐层剥：内层若还是 *url.Error 也一起剥（只剥一层时内层仍带同一段地址）； - *url.Error 的 Err 为 nil（或链自引用）时不再回落到原串，退化成只含 Op 的 替身 —— 那种错误无法在不带地址的前提下原样返回，而地址里就是凭据。 顺带把 httpclient 的 user:pass@ 用例改成变异可打红：net/http 自己在 *url.Error 里就把密码打成 ***，「密码不出现」在剥壳与不剥壳下都成立（空跑）， 用户名才是分水岭。 验收：go build ./... + go vet ./... + 指定包 go test -count=1 全绿； 变异验证（把 URLCause 改成透传）⇒ 五处「不含 URL」断言全红。
59 lines
3.0 KiB
Go
59 lines
3.0 KiB
Go
package trace
|
|
|
|
import (
|
|
"errors"
|
|
"net/url"
|
|
)
|
|
|
|
// 本文件是「从 net/http 的 *url.Error 里取回原因」的**唯一**实现。
|
|
//
|
|
// 为什么需要它：*url.Error 的 Error() 会把**整条地址**渲染进错误串 ——
|
|
// `Get "https://h/p?access_token=…": dial tcp …`。而 net/http 在两条路上都会包出它：
|
|
// 传输层失败（`client.Do`）与地址解析失败（`http.NewRequest`）。这条错误串会进日志、
|
|
// 进工单、进模型可见视图，而平台的资源地址几乎都带凭据（预签名上传地址的
|
|
// `encrypted_query_param`、CDN 的 `rkey`、Bot API 路径里的 `/bot<token>/`、OAuth 回调
|
|
// 里的 `access_token`、user:pass@），于是「原样外抛」等于把凭据写进日志。
|
|
//
|
|
// 为什么不能 `fmt.Errorf("...: %w", err)` 直接外抛：%w 只给错误串加前缀，地址仍在串里
|
|
// —— 包一层不是修复，只是换个位置泄漏。剥壳（取内层 cause）才是。
|
|
//
|
|
// 这段逻辑曾被五个客户端各写一份逐字同形的三行（mediafetch.sanitizeErr /
|
|
// httpclient.transportCause / vision.fetchCause / telegram.callCause /
|
|
// weixin_ilink.urlFreeCause，共 13 个调用点）。同形实现一分家，最容易发生的漂移是
|
|
// 「其中一份为了多带点诊断把地址留下来」，而那正是凭据进日志的那一条路 —— 因此收敛
|
|
// 到这里，调用点只剩一次函数调用（AGENTS.md §10「三次复制必须抽取」）。
|
|
|
|
// URLCause 剥掉 err 里**所有层**的 *url.Error 外壳，返回不携带任何 URL 的内层原因。
|
|
//
|
|
// 三条口径：
|
|
// - **逐层剥**：内层若还是 *url.Error（调用方又包了一次 Do、或链上叠了两层），继续
|
|
// 往下剥 —— 只剥一层的话内层那条仍然带着同一段地址；
|
|
// - **只丢地址、不丢原因**：返回的是原来的底层 error 本身（不新增包装），因此
|
|
// errors.Is/As 对 context.Canceled、context.DeadlineExceeded、*net.DNSError 等
|
|
// 哨兵仍然成立 —— 调用方的分类（重试 / 降级 / 熔断）不受影响，也绝不会把同一个
|
|
// 原因在文案里说两遍；
|
|
// - **非 *url.Error 原样返回**：同一个值、不加包装，可安全当作「可能剥壳、也可能
|
|
// 什么都没做」的透传。
|
|
//
|
|
// 唯一的例外是 `*url.Error` 没有内层原因（Err 为 nil）或链自引用：那种错误**无法**
|
|
// 在不带地址的前提下原样返回，于是退化成只含操作名（Get/Post/parse）的替身 —— 宁可
|
|
// 少一条原因，也不回落到原串（回落就是泄漏）。net/http 的两处构造都填了 Err，因此
|
|
// 这条路在真实调用中到不了。
|
|
func URLCause(err error) error {
|
|
for {
|
|
var uerr *url.Error
|
|
if !errors.As(err, &uerr) {
|
|
return err
|
|
}
|
|
if uerr.Err == nil || uerr.Err == err {
|
|
return urlOpOnly(uerr.Op)
|
|
}
|
|
err = uerr.Err
|
|
}
|
|
}
|
|
|
|
// urlOpOnly 是「*url.Error 没有可用内层原因」时的替身：只留操作名，地址一个字节都不留。
|
|
func urlOpOnly(op string) error {
|
|
return errors.New("url " + op + " failed")
|
|
}
|