已合并
fix(manager): 修复管理面导入导出身份中心地址解析及连接异常处理 #589
Wal1et创建于 17 天前
fix(manager): 修复管理面导入导出身份中心地址解析及连接异常处理 #589
已合并
Pull Request已成功合入, 合并人@曹玉哲
（感谢 Wal1et 的贡献）
atomgit-bot
17 天前 评论：
17 天前 评论：
变更摘要
本 PR 修复管理面导入导出功能在解析身份中心地址及处理连接异常方面的问题。核心是新增 IdentityServiceUnavailableError 异常类型，在 cluster 适配器中通过新的 _identity_base_url 从 settings.identity_public_key_url 解析身份中心基础地址（并回退到 settings.manager_web_idp_target），同时让 _identity_request 在底层 httpx 传输失败时抛出该异常；import_export_routers 的导入/导出相关路由据此统一返回 503，并补充了对应单元测试。
主要改动
- 新增身份中心不可用异常: 在
registry.py中定义IdentityServiceUnavailableError``（继承自RuntimeError`），作为导入导出操作无法连接身份服务时对外传递的异常类型。 - 身份中心地址解析逻辑:
cluster.py新增_identity_base_url，从settings.identity_public_key_url去除/v1/auth/public_key后缀得到基础地址，无法匹配时回退到settings.manager_web_idp_target。 _identity_request连接处理增强: 将请求基础地址切换为_identity_base_url的解析结果，客户端增加trust_env=False，并在捕获httpx.RequestError后抛出IdentityServiceUnavailableError。- 路由层 503 响应统一:
import_export_routers.py的export_resource、export_resources、preflight_import、import_resource均新增对IdentityServiceUnavailableError的捕获，转换为HTTPException(status_code=503)。 - 新增身份服务相关单元测试: 添加
test_import_export_identity.py，覆盖地址解析、请求头/请求体转发、各类传输异常到 503 的映射，以及导入导出路由在身份中心不可用时返回 503 的行为。
不准确？
atomgit-bot
17 天前 评论：
17 天前 评论：
openJiuwen-bot
17 天前 评论：
17 天前 评论：
欢迎来到 openJiuwen 社区
Hey @u010240195 , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用，可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。
atomgit-bot
17 天前 评论：
17 天前 评论：
17 天前 添加了label：openJiuwen-cla/yes
17 天前 关联了issue：[Feature]: Agent Runtime - Agent执行
17 天前 修改了pull request 的描述
openJiuwen-bot
17 天前 评论：
17 天前 评论：
| 任务名称 | 结果 | 日志操作 |
|---|---|---|
| 静态检查 | ❌FAILED | 点此跳转 |
| 禁用词扫描 | ✅SUCCESS | 点此跳转 |
| 防投毒检查 | ❌FAILED | 点此跳转 |
| 开源合规检查 | ✅SUCCESS | 点此跳转 |
| UT测试 | N/A | N/A |
| ST测试 | N/A | N/A |
| build 编译包 | N/A | N/A |
| ruff codecheck | {{ruff_codecheck_result}} | {{ruff_codecheck_link_text}} |
| issue关联检查 | {{issue_linked_result}} | {{issue_linked_link_text}} |
| pr size检查 | {{pr_size_result}} | {{pr_size_link_text}} |
17 天前 添加了label：ci-failed
17 天前 删除了label：openJiuwen-cla/yes
17 天前 添加了label：openJiuwen-cla/yes
openJiuwen-bot
17 天前 评论：
17 天前 评论：
17 天前 删除了label：ci-failed
openJiuwen-bot
17 天前 评论：
17 天前 评论：
| 任务名称 | 结果 | 日志操作 |
|---|---|---|
| 静态检查 | ✅SUCCESS | 点此跳转 |
| 禁用词扫描 | ✅SUCCESS | 点此跳转 |
| 防投毒检查 | ✅SUCCESS | 点此跳转 |
| 开源合规检查 | ✅SUCCESS | 点此跳转 |
| UT测试 | N/A | N/A |
| ST测试 | N/A | N/A |
| build 编译包 | N/A | N/A |
| ruff codecheck | {{ruff_codecheck_result}} | {{ruff_codecheck_link_text}} |
| issue关联检查 | {{issue_linked_result}} | {{issue_linked_link_text}} |
| pr size检查 | {{pr_size_result}} | {{pr_size_link_text}} |
17 天前 添加了label：ci-successful
15 天前 删除了label：ci-successful
Paired: GitHub #258 ↔ GitCode !589
What type of PR is this?
/kind
Self-checklist:（请自检，在[ ]内打上x，我们将检视你的完成情况，否则会导致pr无法合入）