Bug: OAuthClient.execute 方法中的冗余异常抛出

Bắt đầu với api/src/main/java/com/coze/openapi/service/auth/OAuthClient.java, tập trung vào các dòng 228-265 và entry point execute. Xác minh cách các phản hồi không thành công và HttpException được xử lý, loại bỏ phần xử lý dư thừa được mô tả trong issue, rồi chạy các test hiện có của repository để xác nhận rằng các lỗi xác thực vẫn được truyền đi chính xác.

问题描述

在 OAuthClient.java 的 execute 方法中，第 259 行的 throw new CozeAuthException(error, e, e.code(), logID) 存在冗余的异常抛出逻辑。

// 第 228-265 行
protected static <T> T execute(Single<Response<T>> apiCall) {
  try {
    Response<T> response = apiCall.blockingGet();
    if (!response.isSuccessful()) {
      try (ResponseBody errorBody = response.errorBody()) {
        if (errorBody == null) {
          throw new HttpException(response);
        }
        String logID = Utils.getLogID(response);
        String errStr = errorBody.string();
        CozeError error = mapper.readValue(errStr, CozeError.class);
        throw new CozeAuthException(error, new HttpException(response), response.code(), logID); // 第239行：第一次处理已抛出
      } catch (IOException ex) {
        throw new HttpException(response);
      }
    }
    // ...
  } catch (HttpException e) { // 第249行
    try (ResponseBody errorBody = e.response().errorBody()) {
      if (errorBody == null) {
        throw e;
      }
      String logID = Utils.getLogID(e.response());
      String errStr = e.response().errorBody().string();
      CozeError error = mapper.readValue(errStr, CozeError.class);
      throw new CozeAuthException(error, e, e.code(), logID); // 第259行：冗余的第二次处理
    } catch (IOException ex) {
      throw e;
    }
  }
}

问题分析

1. 第 239 行已经对非成功响应进行了错误解析并抛出 CozeAuthException，而第 249-263 行的 catch (HttpException e) 块又再次进行了相同的错误解析并抛出 CozeAuthException。

2. 第 259 行的 throw new CozeAuthException 会导致应用无法正常捕获该异常。

3. 当第一次解析成功时，第239行的 CozeAuthException 会直接传播出去，不会进入第249行的捕获块；

建议修改

去掉第 249-263 行的 catch (HttpException e) 块，去掉第259行的 throw new CozeAuthException(error, e, e.code(), logID)，因为这段代码是冗余的。第一次处理已经足够，不需要重复解析。

影响版本

影响所有版本

文件位置

api/src/main/java/com/coze/openapi/service/auth/OAuthClient.java 第 249-263 行