Dev multi tenant

[wyyalt]

No description provided.

[github-actions]

🔍 代码审查总结

审查了本次 PR 的核心变更，发现 6 个关键问题需要处理：

🔒 安全问题（必须修复）

1. 认证检查被注释 - gcloud/utils/middleware.py:42-43
   • 允许未验证请求绕过认证
2. 敏感日志泄漏 - gcloud/core/middlewares.py:160-161
   • 使用 logger.error() 输出完整请求元数据

⚡ 破坏性变更

3. API 签名改变 - api/utils/request.py:37
   • batch_request 函数调用方式变更，需确认所有调用方已更新

⚠️ 逻辑风险

4. 租户 ID 弱回退 - gcloud/core/middlewares.py:171-173
   • 缺失 tenant_id 时默认为 "default"，可能导致跨租户数据泄漏
5. 缓存键不安全 - gcloud/core/middlewares.py:41
   • 时区缓存键未包含 tenant_id，可能返回错误租户的数据
6. IAM 环境配置 - config/default.py:767
   • 默认连接生产环境，测试环境可能误调用

---

建议： 问题 #1 和 #2 为高危安全问题，建议在合并前修复。其他问题请评估影响范围后处理。

详细评论已添加到对应代码行。

[github-actions]

File: gcloud/utils/middleware.py
Line: 42-43

🔒 CRITICAL: 认证检查被注释

# if not verified:
#     return self.make_anonymous_user(bk_username=bk_username)

问题： 所有未验证的请求都能绕过认证，被当作已认证用户处理

修复： 取消注释此安全检查，或提供明确文档说明为何禁用

[github-actions]

File: gcloud/core/middlewares.py
Line: 160-161

🚨 敏感信息日志泄漏

logger.error("+++++++++++{}: {}++++++++++".format(header, header_value))
logger.error("+++++++++++request meta: {}++++++++++".format(request.META))

问题： 生产环境日志会包含敏感 header、token、IP 地址

修复： 删除这些调试日志，或改用 logger.debug()

[github-actions]

File: gcloud/core/middlewares.py
Line: 171-173

⚠️ 租户 ID 弱回退逻辑

if not tenant_id:
    tenant_id = "default"

问题： 缺失 tenant_id 时静默回退到 "default"，可能导致跨租户数据访问

修复： 在多租户模式下，缺失 tenant_id 应返回 403 或抛出异常

[github-actions]

File: gcloud/core/middlewares.py
Line: 41

⚠️ 缓存键缺少租户隔离

user_time_zone_cache_key = f"{request.user.username}_time_zone"

问题： 租户 X 的用户 A 可能获取到租户 Y 的用户 A 的时区设置

修复： 包含 tenant_id：f"{request.user.tenant_id}_{request.user.username}_time_zone"

[github-actions]

File: api/utils/request.py
Line: 37

⚡ 破坏性 API 变更

- return target_func(**request_params)
+ return target_func(request_params, path_params=path_params, headers=headers)

问题： 函数签名从关键字参数变为位置参数，所有调用方需要更新

修复： 确认所有调用代码已更新，或保持向后兼容