team/tyapi-server
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix

Browse Source
This commit is contained in:
liangzai
2025-08-27 22:19:19 +08:00
parent 4031277a91
commit 5051aea55c
93 changed files with 2025 additions and 1168 deletions
Download Patch File Download Diff File Expand all files Collapse all files

123
internal/infrastructure/external/README.md vendored Normal file
View File

@@ -0,0 +1,123 @@
# 外部服务错误处理修复说明
## 问题描述
在外部服务（WestDex、Yushan、Zhicha）中，使用 `fmt.Errorf("%w: %s", ErrXXX, err)` 包装错误后，外层的 `errors.Is(err, ErrXXX)` 无法正确识别错误类型。
## 问题原因
`fmt.Errorf` 创建的包装错误虽然实现了 `Unwrap()` 接口，但没有实现 `Is()` 接口，因此 `errors.Is` 无法正确判断错误类型。
## 修复方案
统一使用 `errors.Join` 来组合错误，这是 Go 1.20+ 的标准做法，天然支持 `errors.Is` 判断。
## 修复内容
### 1. WestDex 服务 (`westdex_service.go`)
#### 修复前：
```go
// 无法被 errors.Is 识别的错误包装
err = fmt.Errorf("%w: %s", ErrSystem, marshalErr.Error())
err = fmt.Errorf("%w: %s", ErrDatasource, westDexResp.Message)
```
#### 修复后：
```go
// 可以被 errors.Is 正确识别的错误组合
err = errors.Join(ErrSystem, marshalErr)
err = errors.Join(ErrDatasource, fmt.Errorf(westDexResp.Message))
```
### 2. Yushan 服务 (`yushan_service.go`)
#### 修复前：
```go
// 无法被 errors.Is 识别的错误包装
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = fmt.Errorf("%w: %s", ErrDatasource, "羽山请求retdata为空")
```
#### 修复后：
```go
// 可以被 errors.Is 正确识别的错误组合
err = errors.Join(ErrSystem, err)
err = errors.Join(ErrDatasource, fmt.Errorf("羽山请求retdata为空"))
```
### 3. Zhicha 服务 (`zhicha_service.go`)
#### 修复前：
```go
// 无法被 errors.Is 识别的错误包装
err = fmt.Errorf("%w: %s", ErrSystem, marshalErr.Error())
err = fmt.Errorf("%w: %s", ErrDatasource, "HTTP状态码 %d", response.StatusCode)
```
#### 修复后：
```go
// 可以被 errors.Is 正确识别的错误组合
err = errors.Join(ErrSystem, marshalErr)
err = errors.Join(ErrDatasource, fmt.Errorf("HTTP状态码 %d", response.StatusCode))
```
## 修复效果
### 修复前的问题：
```go
// 在应用服务层
if errors.Is(err, westdex.ErrDatasource) {
// 这里无法正确识别，因为 fmt.Errorf 包装的错误
// 没有实现 Is() 接口
return ErrDatasource
}
```
### 修复后的效果：
```go
// 在应用服务层
if errors.Is(err, westdex.ErrDatasource) {
// 现在可以正确识别了！
return ErrDatasource
}
if errors.Is(err, westdex.ErrSystem) {
// 系统错误也能正确识别
return ErrSystem
}
```
## 优势
1. **完全兼容**：`errors.Is` 现在可以正确识别所有错误类型
2. **标准做法**：使用 Go 1.20+ 的 `errors.Join` 标准库功能
3. **性能优秀**：标准库实现，性能优于自定义解决方案
4. **维护简单**：无需自定义错误类型，代码更简洁
## 注意事项
1. **Go版本要求**：需要 Go 1.20 或更高版本（项目使用 Go 1.23.4，完全满足）
2. **错误消息格式**：`errors.Join` 使用换行符分隔多个错误
3. **向后兼容**：现有的错误处理代码无需修改
## 测试验证
所有修复后的外部服务都能正确编译：
```bash
go build ./internal/infrastructure/external/westdex/...
go build ./internal/infrastructure/external/yushan/...
go build ./internal/infrastructure/external/zhicha/...
```
## 总结
通过统一使用 `errors.Join` 修复外部服务的错误处理，现在：
- ✅ `errors.Is(err, ErrDatasource)` 可以正确识别数据源异常
- ✅ `errors.Is(err, ErrSystem)` 可以正确识别系统异常
- ✅ `errors.Is(err, ErrNotFound)` 可以正确识别查询为空
- ✅ 错误处理逻辑更加清晰和可靠
- ✅ 符合 Go 1.20+ 的最佳实践
这个修复确保了整个系统的错误处理链路都能正确工作，提高了系统的可靠性和可维护性。

48
internal/infrastructure/external/westdex/westdex_service.go vendored
View File

@@ -99,7 +99,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
jsonData, marshalErr := json.Marshal(reqData)
if marshalErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, marshalErr.Error())
err = errors.Join(ErrSystem, marshalErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -109,7 +109,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
// 创建HTTP POST请求
req, newRequestErr := http.NewRequestWithContext(ctx, "POST", reqUrl, bytes.NewBuffer(jsonData))
if newRequestErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, newRequestErr.Error())
err = errors.Join(ErrSystem, newRequestErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -123,7 +123,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
client := &http.Client{}
httpResp, clientDoErr := client.Do(req)
if clientDoErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, clientDoErr.Error())
err = errors.Join(ErrSystem, clientDoErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -134,7 +134,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
if closeErr != nil {
// 记录关闭错误
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, fmt.Errorf("关闭响应体失败: %w", closeErr), reqData)
w.logger.LogError(requestID, transactionID, code, errors.Join(ErrSystem, fmt.Errorf("关闭响应体失败: %w", closeErr)), reqData)
}
}
}(httpResp.Body)
@@ -147,7 +147,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
// 读取响应体
bodyBytes, ReadErr := io.ReadAll(httpResp.Body)
if ReadErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, ReadErr.Error())
err = errors.Join(ErrSystem, ReadErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -158,7 +158,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
var westDexResp WestResp
UnmarshalErr := json.Unmarshal(bodyBytes, &westDexResp)
if UnmarshalErr != nil {
err = UnmarshalErr
err = errors.Join(ErrSystem, UnmarshalErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -172,7 +172,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
if westDexResp.Code != "00000" && westDexResp.Code != "200" && westDexResp.Code != "0" {
if westDexResp.Data == "" {
err = fmt.Errorf("%w: %s", ErrSystem, westDexResp.Message)
err = errors.Join(ErrSystem, fmt.Errorf(westDexResp.Message))
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -180,7 +180,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
}
decryptedData, DecryptErr := crypto.WestDexDecrypt(westDexResp.Data, w.config.Key)
if DecryptErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, DecryptErr.Error())
err = errors.Join(ErrSystem, DecryptErr)
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -189,17 +189,17 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
// 记录业务错误日志，包含响应ID
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, fmt.Errorf("%w: %s", ErrDatasource, westDexResp.Message), reqData, westDexResp.ID)
w.logger.LogErrorWithResponseID(requestID, transactionID, code, errors.Join(ErrDatasource, fmt.Errorf(westDexResp.Message)), reqData, westDexResp.ID)
}
// 记录性能日志（失败）
// 注意：通用日志系统不包含性能日志功能
return decryptedData, fmt.Errorf("%w: %s", ErrDatasource, westDexResp.Message)
return decryptedData, errors.Join(ErrDatasource, fmt.Errorf(westDexResp.Message))
}
if westDexResp.Data == "" {
err = fmt.Errorf("%w: %s", ErrSystem, westDexResp.Message)
err = errors.Join(ErrSystem, fmt.Errorf(westDexResp.Message))
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -208,7 +208,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
decryptedData, DecryptErr := crypto.WestDexDecrypt(westDexResp.Data, w.config.Key)
if DecryptErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, DecryptErr.Error())
err = errors.Join(ErrSystem, DecryptErr)
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -222,7 +222,7 @@ func (w *WestDexService) CallAPI(ctx context.Context, code string, reqData map[s
}
// 记录HTTP错误
err = fmt.Errorf("%w: 西部请求失败Code: %d", ErrSystem, httpResp.StatusCode)
err = errors.Join(ErrSystem, fmt.Errorf("西部请求失败Code: %d", httpResp.StatusCode))
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
// 注意：通用日志系统不包含性能日志功能
@@ -252,7 +252,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
jsonData, marshalErr := json.Marshal(reqData)
if marshalErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, marshalErr.Error())
err = errors.Join(ErrSystem, marshalErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -262,7 +262,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
// 创建HTTP POST请求
req, newRequestErr := http.NewRequestWithContext(ctx, "POST", reqUrl, bytes.NewBuffer(jsonData))
if newRequestErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, newRequestErr.Error())
err = errors.Join(ErrSystem, newRequestErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -276,7 +276,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
client := &http.Client{}
httpResp, clientDoErr := client.Do(req)
if clientDoErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, clientDoErr.Error())
err = errors.Join(ErrSystem, clientDoErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -287,7 +287,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
if closeErr != nil {
// 记录关闭错误
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, fmt.Errorf("关闭响应体失败: %w", closeErr), reqData)
w.logger.LogError(requestID, transactionID, code, errors.Join(ErrSystem, fmt.Errorf("关闭响应体失败: %w", closeErr)), reqData)
}
}
}(httpResp.Body)
@@ -298,7 +298,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
if httpResp.StatusCode == 200 {
bodyBytes, ReadErr := io.ReadAll(httpResp.Body)
if ReadErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, ReadErr.Error())
err = errors.Join(ErrSystem, ReadErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -308,7 +308,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
var westDexResp G05HZ01WestResp
UnmarshalErr := json.Unmarshal(bodyBytes, &westDexResp)
if UnmarshalErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, UnmarshalErr.Error())
err = errors.Join(ErrSystem, UnmarshalErr)
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
}
@@ -322,7 +322,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
if westDexResp.Code != "0000" {
if westDexResp.Data == nil {
err = fmt.Errorf("%w: %s", ErrSystem, westDexResp.Message)
err = errors.Join(ErrSystem, fmt.Errorf(westDexResp.Message))
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -330,18 +330,18 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
} else {
// 记录业务错误日志，包含响应ID
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, fmt.Errorf("%w: %s", ErrSystem, string(westDexResp.Data)), reqData, westDexResp.ID)
w.logger.LogErrorWithResponseID(requestID, transactionID, code, errors.Join(ErrSystem, fmt.Errorf(string(westDexResp.Data))), reqData, westDexResp.ID)
}
// 记录性能日志（失败）
// 注意：通用日志系统不包含性能日志功能
return westDexResp.Data, fmt.Errorf("%w: %s", ErrSystem, string(westDexResp.Data))
return westDexResp.Data, errors.Join(ErrSystem, fmt.Errorf(string(westDexResp.Data)))
}
}
if westDexResp.Data == nil {
err = fmt.Errorf("%w: %s", ErrSystem, westDexResp.Message)
err = errors.Join(ErrSystem, fmt.Errorf(westDexResp.Message))
if w.logger != nil {
w.logger.LogErrorWithResponseID(requestID, transactionID, code, err, reqData, westDexResp.ID)
}
@@ -354,7 +354,7 @@ func (w *WestDexService) G05HZ01CallAPI(ctx context.Context, code string, reqDat
return westDexResp.Data, nil
} else {
// 记录HTTP错误
err = fmt.Errorf("%w: 西部请求失败Code: %d", ErrSystem, httpResp.StatusCode)
err = errors.Join(ErrSystem, fmt.Errorf("西部请求失败Code: %d", httpResp.StatusCode))
if w.logger != nil {
w.logger.LogError(requestID, transactionID, code, err, reqData)
// 注意：通用日志系统不包含性能日志功能

14
internal/infrastructure/external/yushan/yushan_service.go vendored
View File

@@ -84,7 +84,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
// 将请求数据转换为 JSON 字节数组
messageBytes, err := json.Marshal(reqData)
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -94,7 +94,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
// 获取 API 密钥
key, err := hex.DecodeString(y.config.ApiKey)
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -113,7 +113,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
}
req, err := http.NewRequestWithContext(ctx, "POST", y.config.URL, strings.NewReader(content))
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -125,7 +125,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
// 执行请求
resp, err := client.Do(req)
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -149,7 +149,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
} else {
sDec, err := base64.StdEncoding.DecodeString(string(body))
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -172,7 +172,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
// retcode 为 000000，表示有数据，返回 retdata
retData := gjson.GetBytes(respData, "retdata")
if !retData.Exists() {
err = fmt.Errorf("%w: %s", ErrDatasource, "羽山请求retdata为空")
err = errors.Join(ErrDatasource, fmt.Errorf("羽山请求retdata为空"))
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}
@@ -180,7 +180,7 @@ func (y *YushanService) CallAPI(ctx context.Context, code string, params map[str
}
return []byte(retData.Raw), nil
} else {
err = fmt.Errorf("%w: %s", ErrDatasource, "羽山请求未知的状态码")
err = errors.Join(ErrDatasource, fmt.Errorf("羽山请求未知的状态码"))
if y.logger != nil {
y.logger.LogError(requestID, transactionID, code, err, params)
}

14
internal/infrastructure/external/zhicha/zhicha_service.go vendored
View File

@@ -34,7 +34,7 @@ type ZhichaResp struct {
type ZhichaConfig struct {
URL string
AppID string
AppSecret string
AppSecret string
EncryptKey string
}
@@ -94,7 +94,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
jsonData, marshalErr := json.Marshal(params)
if marshalErr != nil {
err = fmt.Errorf("%w: %s", ErrSystem, marshalErr.Error())
err = errors.Join(ErrSystem, marshalErr)
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}
@@ -104,7 +104,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
// 创建HTTP POST请求
req, err := http.NewRequestWithContext(ctx, "POST", z.config.URL, bytes.NewBuffer(jsonData))
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}
@@ -126,7 +126,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
// 发送请求
response, err := client.Do(req)
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}
@@ -137,7 +137,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
// 读取响应
respBody, err := io.ReadAll(response.Body)
if err != nil {
err = fmt.Errorf("%w: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, err)
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}
@@ -152,7 +152,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
// 检查HTTP状态码
if response.StatusCode != http.StatusOK {
err = fmt.Errorf("%w: HTTP状态码 %d", ErrDatasource, response.StatusCode)
err = errors.Join(ErrDatasource, fmt.Errorf("HTTP状态码 %d", response.StatusCode))
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}
@@ -162,7 +162,7 @@ func (z *ZhichaService) CallAPI(ctx context.Context, proID string, params map[st
// 解析响应
var zhichaResp ZhichaResp
if err := json.Unmarshal(respBody, &zhichaResp); err != nil {
err = fmt.Errorf("%w: 响应解析失败: %s", ErrSystem, err.Error())
err = errors.Join(ErrSystem, fmt.Errorf("响应解析失败: %s", err.Error()))
if z.logger != nil {
z.logger.LogError(requestID, transactionID, proID, err, params)
}