1fec510e2e
## 主要完成内容
### 1. 第一优先级模块迁移
- startup.py:19个端点(伦理审批、可行性评估、预算、时间表)
- project_permissions.py:2个端点(项目权限查询、更新)
- overview.py:1个端点(项目概览)
### 2. 第二优先级模块迁移
- monitoring_visit_issues.py:7个端点(监查问题管理)
- drug_shipments.py:5个端点(药物发货管理)
- material_equipments.py:5个端点(物资管理)
- subject_pds.py:4个端点(参与者PDS)
- audit_logs.py:3个端点(审计日志)
### 3. 第三优先级模块迁移
- visits.py:5个端点(访视管理)
- knowledge_notes.py:5个端点(知识库笔记)
- subject_histories.py:5个端点(参与者历史)
- project_milestones.py:2个端点(项目里程碑)
### 4. 测试与验证
- 34个集成测试全部通过
- 13个配置测试全部通过
- 109个总测试全部通过
- 代码覆盖率≥80%
### 5. 文档更新
- 更新TESTING_SUMMARY.md
- 更新IMPLEMENTATION_SUMMARY.md
## 统计数据
- 迁移端点数:63个
- 迁移模块数:12个
- 新增测试:34个
- 总测试数:109个
- 代码覆盖率:85%
- 已迁移端点总数:94个(第1-3批)
## 权限配置修复
修复了api_permissions.py中的模块名称映射,确保所有63个端点的模块名称与DEFAULT_PROJECT_ROLE_PERMISSIONS中的有效模块名称一致。
## 测试修复
修复了test_api_permissions_config.py中的两个测试用例:
- test_project_members_endpoints_configured:更新为使用正确的/studies/{study_id}/members路径
- test_sites_endpoints_configured:更新为使用正确的/studies/{study_id}/sites路径
Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
274 lines
8.2 KiB
Markdown
274 lines
8.2 KiB
Markdown
# 接口级权限系统 - 实现总结
|
||
|
||
## 项目概述
|
||
|
||
本项目实现了从模块级权限到接口级权限的系统迁移,支持更细粒度的API端点级权限控制,同时保持向后兼容性。
|
||
|
||
## 实现阶段
|
||
|
||
### 第1-6阶段:核心基础设施(已完成)
|
||
|
||
#### 1. 数据库模型
|
||
- **ApiEndpointPermission**:存储接口级权限配置
|
||
- **ApiEndpointRegistry**:注册系统中所有API端点
|
||
|
||
#### 2. 权限配置系统
|
||
- **api_permissions.py**:中央权限配置注册表
|
||
- **MODULE_TO_ENDPOINTS**:向后兼容映射表
|
||
|
||
#### 3. 权限检查函数
|
||
- **role_has_api_permission()**:检查角色是否有权访问特定接口
|
||
- **get_api_endpoint_permissions()**:获取项目权限矩阵
|
||
- **replace_api_endpoint_permissions()**:更新项目权限矩阵
|
||
|
||
#### 4. FastAPI 集成
|
||
- **require_api_permission()**:依赖注入函数
|
||
- **@register_api_endpoint**:装饰器注册端点
|
||
|
||
#### 5. 权限管理API
|
||
- **GET /studies/{study_id}/api-permissions**:获取权限矩阵
|
||
- **PUT /studies/{study_id}/api-permissions**:更新权限矩阵
|
||
|
||
### 第8阶段:迁移第3批模块(已完成)
|
||
|
||
#### 迁移的模块
|
||
|
||
**第一优先级(关键业务)**
|
||
- startup.py:19个端点(伦理审批、可行性评估、预算、时间表)
|
||
- project_permissions.py:2个端点(项目权限查询、更新)
|
||
- overview.py:1个端点(项目概览)
|
||
|
||
**第二优先级(重要业务)**
|
||
- monitoring_visit_issues.py:7个端点(监查问题管理)
|
||
- drug_shipments.py:5个端点(药物发货管理)
|
||
- material_equipments.py:5个端点(物资管理)
|
||
- subject_pds.py:4个端点(参与者PDS)
|
||
- audit_logs.py:3个端点(审计日志)
|
||
|
||
**第三优先级(辅助功能)**
|
||
- visits.py:5个端点(访视管理)
|
||
- knowledge_notes.py:5个端点(知识库笔记)
|
||
- subject_histories.py:5个端点(参与者历史)
|
||
- project_milestones.py:2个端点(项目里程碑)
|
||
|
||
#### 迁移统计
|
||
- **迁移端点数**:63 个
|
||
- **涉及文件**:12 个
|
||
- **新增测试**:34 个
|
||
- **总测试数**:109 个(包括前7阶段的75个)
|
||
|
||
## 权限配置示例
|
||
|
||
### Members 模块权限矩阵
|
||
|
||
| 角色 | 添加成员 | 查询成员 | 更新成员 | 删除成员 | 查询候选人 |
|
||
|------|---------|---------|---------|---------|-----------|
|
||
| ADMIN | ✅ | ✅ | ✅ | ✅ | ✅ |
|
||
| PM | ✅ | ✅ | ✅ | ✅ | ✅ |
|
||
| CRA | ❌ | ❌ | ❌ | ❌ | ❌ |
|
||
| PV | ❌ | ❌ | ❌ | ❌ | ❌ |
|
||
| MEDICAL_REVIEW | ❌ | ❌ | ❌ | ❌ | ❌ |
|
||
| IMP | ❌ | ❌ | ❌ | ❌ | ❌ |
|
||
| QA | ❌ | ❌ | ❌ | ❌ | ❌ |
|
||
|
||
### Sites 模块权限矩阵
|
||
|
||
| 角色 | 创建中心 | 查询中心 | 更新中心 | 删除中心 |
|
||
|------|---------|---------|---------|---------|
|
||
| ADMIN | ✅ | ✅ | ✅ | ✅ |
|
||
| PM | ✅ | ✅ | ✅ | ✅ |
|
||
| CRA | ❌ | ❌ | ❌ | ❌ |
|
||
| PV | ❌ | ❌ | ❌ | ❌ |
|
||
| MEDICAL_REVIEW | ❌ | ❌ | ❌ | ❌ |
|
||
| IMP | ❌ | ❌ | ❌ | ❌ |
|
||
| QA | ❌ | ❌ | ❌ | ❌ |
|
||
|
||
## 迁移代码示例
|
||
|
||
### 迁移前(模块级权限)
|
||
```python
|
||
@router.post(
|
||
"/",
|
||
response_model=StudyMemberRead,
|
||
dependencies=[
|
||
Depends(require_study_permission("project_members", "write")),
|
||
],
|
||
)
|
||
async def add_member(...):
|
||
pass
|
||
```
|
||
|
||
### 迁移后(接口级权限)
|
||
```python
|
||
@router.post(
|
||
"/",
|
||
response_model=StudyMemberRead,
|
||
dependencies=[
|
||
Depends(require_api_permission("POST:/studies/{study_id}/members")),
|
||
Depends(require_study_not_locked())
|
||
],
|
||
)
|
||
@register_api_endpoint(
|
||
endpoint_key="POST:/studies/{study_id}/members",
|
||
module="project_members",
|
||
action="write",
|
||
description="添加项目成员",
|
||
default_roles=["PM"],
|
||
)
|
||
async def add_member(...):
|
||
pass
|
||
```
|
||
|
||
## 权限检查流程
|
||
|
||
```
|
||
请求到达
|
||
↓
|
||
FastAPI依赖注入 → require_api_permission("POST:/studies/{study_id}/members")
|
||
↓
|
||
role_has_api_permission(db, study_id, role, "POST:/studies/{study_id}/members")
|
||
↓
|
||
├─ 查询 ApiEndpointPermission 表
|
||
│ ├─ 找到 → 返回 allowed 值
|
||
│ └─ 未找到 → 继续
|
||
│
|
||
└─ 回退到模块级权限
|
||
└─ role_has_project_permission(db, study_id, role, "project_members", "write")
|
||
├─ 查询 StudyRolePermission 表
|
||
└─ 返回权限结果
|
||
↓
|
||
权限检查通过 → 执行业务逻辑
|
||
权限检查失败 → 返回 403 Forbidden
|
||
```
|
||
|
||
## 已迁移端点总览
|
||
|
||
### 第1批(22个端点)
|
||
- **subjects**:5 个端点
|
||
- **risk_issues**:3 个端点
|
||
- **fees**:8 个端点
|
||
- **finance_contracts**:5 个端点
|
||
|
||
### 第2批(9个端点)
|
||
- **members**:5 个端点
|
||
- **sites**:4 个端点
|
||
|
||
### 第3批(63个端点)
|
||
- **startup**:19 个端点
|
||
- **project_permissions**:2 个端点
|
||
- **overview**:1 个端点
|
||
- **monitoring_visit_issues**:7 个端点
|
||
- **drug_shipments**:5 个端点
|
||
- **material_equipments**:5 个端点
|
||
- **subject_pds**:4 个端点
|
||
- **audit_logs**:3 个端点
|
||
- **visits**:5 个端点
|
||
- **knowledge_notes**:5 个端点
|
||
- **subject_histories**:5 个端点
|
||
- **project_milestones**:2 个端点
|
||
|
||
### 总计:94 个端点
|
||
|
||
## 向后兼容性
|
||
|
||
系统支持两种权限检查方式的并行运行:
|
||
|
||
1. **接口级权限**(优先级高)
|
||
- 存储在 `ApiEndpointPermission` 表
|
||
- 支持细粒度的端点级控制
|
||
|
||
2. **模块级权限**(优先级低)
|
||
- 存储在 `StudyRolePermission` 表
|
||
- 用于未迁移的端点和向后兼容
|
||
|
||
**优先级规则**:
|
||
- 如果存在接口级权限配置,使用接口级权限
|
||
- 如果不存在接口级权限配置,回退到模块级权限
|
||
- 如果两者都不存在,拒绝访问
|
||
|
||
## 测试覆盖
|
||
|
||
### 测试文件
|
||
- `test_api_permissions.py`:12 个测试
|
||
- `test_api_permissions_endpoints.py`:11 个测试
|
||
- `test_api_permissions_config.py`:13 个测试
|
||
- `test_migrated_endpoints.py`:22 个测试(第1批)
|
||
- `test_migrated_endpoints_batch2.py`:17 个测试(第2批)
|
||
- `test_migrated_endpoints_batch3.py`:34 个测试(第3批)
|
||
|
||
### 测试场景
|
||
- ✅ 接口级权限允许/拒绝
|
||
- ✅ 模块级权限回退
|
||
- ✅ 权限优先级验证
|
||
- ✅ 权限矩阵操作
|
||
- ✅ 向后兼容性验证
|
||
- ✅ 权限隔离验证
|
||
- ✅ 权限配置验证
|
||
|
||
### 覆盖率
|
||
- **代码覆盖率**:85%
|
||
- **测试通过率**:100%(109/109)
|
||
|
||
## 关键文件清单
|
||
|
||
### 新增文件
|
||
| 文件 | 用途 |
|
||
|------|------|
|
||
| `app/models/api_endpoint_permission.py` | 接口级权限模型 |
|
||
| `app/models/api_endpoint_registry.py` | 接口注册表模型 |
|
||
| `app/core/api_permissions.py` | 接口权限配置 |
|
||
| `app/core/decorators.py` | 装饰器辅助函数 |
|
||
| `app/api/v1/api_permissions.py` | 权限管理API |
|
||
| `tests/test_api_permissions.py` | 权限检查测试 |
|
||
| `tests/test_api_permissions_endpoints.py` | 权限管理API测试 |
|
||
| `tests/test_migrated_endpoints.py` | 第1批端点测试 |
|
||
| `tests/test_migrated_endpoints_batch2.py` | 第2批端点测试 |
|
||
|
||
### 修改文件
|
||
| 文件 | 修改内容 |
|
||
|------|---------|
|
||
| `app/core/project_permissions.py` | 新增接口级权限检查函数 |
|
||
| `app/core/deps.py` | 新增 `require_api_permission()` 函数 |
|
||
| `app/api/v1/subjects.py` | 迁移到接口级权限 |
|
||
| `app/api/v1/aes.py` | 迁移到接口级权限 |
|
||
| `app/api/v1/fees_contracts.py` | 迁移到接口级权限 |
|
||
| `app/api/v1/members.py` | 迁移到接口级权限(第2批) |
|
||
| `app/api/v1/sites.py` | 迁移到接口级权限(第2批) |
|
||
|
||
## 性能指标
|
||
|
||
- **测试执行时间**:0.40 秒
|
||
- **平均单个测试时间**:3.7 毫秒
|
||
- **代码覆盖率**:85%
|
||
- **总测试数**:109 个
|
||
|
||
## 下一步工作
|
||
|
||
### 第9阶段:安全审计与性能优化
|
||
**目标模块**:
|
||
- 安全审计:检查权限系统的安全性
|
||
- 性能优化:优化权限检查的性能
|
||
- 缓存策略:实现权限缓存
|
||
|
||
**预计工作量**:4-6 小时
|
||
|
||
### 第10-11阶段
|
||
- 性能测试
|
||
- 文档更新
|
||
|
||
## 总结
|
||
|
||
接口级权限系统已成功实现,包括:
|
||
- ✅ 核心基础设施(数据库、配置、检查函数)
|
||
- ✅ FastAPI 集成(依赖注入、装饰器)
|
||
- ✅ 权限管理API
|
||
- ✅ 第1批模块迁移(22 个端点)
|
||
- ✅ 第2批模块迁移(9 个端点)
|
||
- ✅ 第3批模块迁移(63 个端点)
|
||
- ✅ 全面的测试覆盖(109 个测试)
|
||
- ✅ 向后兼容性保证
|
||
|
||
**已迁移端点总数:94 个**
|
||
|
||
系统已准备好进行第9阶段的安全审计和性能优化。
|