wanglei/EchoChat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
205 Commits 1 Branch 0 Tags
1f4a0da8d409a2b9833b957d7f7576c8f847d177
Commit Graph

11 Commits

Author SHA1 Message Date
duoaohui
1f4a0da8d4 视频会议 2026-05-22 09:17:14 +08:00
duoaohui
e7921556e1 视频会议 2026-05-22 01:48:14 +08:00
duoaohui
19aac86f48 签入签出 2026-05-22 01:02:34 +08:00
duoaohui
fc20a97442 视频会议 2026-05-21 19:01:25 +08:00
bujinyuan
c35097a0d8 fix(meeting): Task 16 修复 code-reviewer 审计 P2 七项 + Nit 七项 
覆盖 Phase 2e-2 代码审查报告（docs/reviews/2026-04-23-phase2e-2-code-review.md）
P2/Nit 收尾批次，均在本仓库完成闭环；剩余 5 项登记推迟至 Phase 2f/3。

===== P2 七项 =====
- P2-1 cleanupUserResources 补 transport 清理
  · media-server 新增 DELETE /internal/v1/transports/:id + transport.service.closeTransport
  · Go MediaOrchestrator 接口新增 CloseTransport；HTTP 实现按 doCloseRequest 走 4xx 幂等 + 指数退避
  · meeting_signal_service.cleanupUserResources 新增 "transport:<id>" 分支
- P2-2 preview.vue 快速切设备竞态
  · previewSeq 序号 + nextTick 后 stale 判断，丢弃过期结果
  · onVideoChange/onAudioChange 走 scheduleRestartPreview 200ms 防抖
  · onBeforeUnmount 清理 changeDebounceTimer
- P2-3 room.vue onLoad redirectTo 后补 return
  · 引入 redirectingToJoin 守卫，跳转页不再执行 onMounted 初始化
- P2-6 generateUniqueRoomCode 重试上限监控
  · 重试后成功：Warn 日志（码空间健康度告警）
  · 重试耗尽：Error 日志 + ErrRoomCodeConflict
  · 修正 logs.Error 调用签名（去掉多余的 nil）
- P2-7 SendChatMessage 服务端长度 + 频率限制
  · 新增 ErrChatContentEmpty / ErrChatContentTooLong / ErrChatRateLimited
  · utf8.RuneCountInString 校验 500 字符上限
  · Redis INCR + EXPIRE 滑动窗口（30 条/60s，首次写入 EXPIRE 兜底）
  · controller.handleError 映射为 HTTP 400
- P2-8 MEETING_ENDED_REASON_LABEL 覆盖复核
  · 新增前端专属常量 MEETING_ENDED_REASON_KICKED + 文案
  · store/meeting.js _onMemberKicked 使用常量
  · 同步修复后端 OnWSDisconnect 硬编码 "ws_disconnect" → constants.MeetingLeftReasonDisconnect
- P2 已修 P2-1/2/3/6/7/8；P2-4（WS token 迁出 URL query）与 P2-5（Chat 服务拆分）登记推迟

===== Nit 七项 =====
- Nit: kind:id 解析改用 strings.SplitN
  · cleanupUserResources / pushExistingRoomState 两处同步
- Nit: resourceTTL 中央化
  · 新增 constants.MeetingResourceTrackTTLSeconds（3600）
  · meeting_signal_service.go resourceTTL 由 const 改 var 并引用常量
- Nit: ws/handler.go CheckOrigin 白名单
  · NewHandler 新增 serverCfg 依赖；checkOrigin 支持同源放行 / dev 模式放行 / release 模式白名单严格匹配
  · config.ServerConfig 新增 WSAllowedOrigins（逗号分隔）+ AllowedOrigins() / IsRelease() 辅助方法
  · provider.go 新增 provideServerConfig，wire_gen.go 同步
- Nit: http_media_orchestrator.go 超时 + CreateRouter 重试
  · 默认 TimeoutMS 5000→10000ms 兼容 Worker 冷启动
  · 新增 CreateRouterRetry（默认 1 次，300ms 退避），仅对非 404 错误重试
  · config.dev.yaml / config.docker.yaml 同步写入显式配置
- Nit: deploy-public.sh REDIS_PASSWORD × redis.conf 联动校验
  · 检测 REDIS_PASSWORD 与 redis.conf 的 requirepass 配对一致性
  · redis.conf 增加公网部署 requirepass 使用说明
- Nit: media-server internal-auth isPrivatePath 按 path 匹配
  · 剔除 query/hash 后再与白名单 startsWith，避免 "?" 语义混淆
- Nit: mediasoup-client.js in-flight 锁走读确认
  · finally 分支已覆盖 resolve/reject 两路，追加注释强化语义
- Nit 走读复核：_onMemberLeft 整槽关闭 vs _onProducerNew(closed=true) 精确匹配 producerId
  · 粒度正确，无需改动（登记结论）

===== 构建验证 =====
- go vet ./... / go build ./... 通过
- frontend npm run build:h5 通过（仅 uni-app legacy warning，无 error）
- media-server npx tsc --noEmit 通过

===== 审查追踪小节 =====
docs/reviews/2026-04-23-phase2e-2-code-review.md 追加 "Task 16 修复追踪（2026-04-24 更新）"：
- 已修复一览（本批次 14 处 + 历次 commit cdaa39d / ea2bf96 / f5ae095 / 5ed14c2）
- 推迟登记表（P2-4 / P2-5 / 端口收敛 / appData 校验 / RFC3339 时间格式，共 5 项）

Made-with: Cursor
 2026-04-23 17:45:03 +08:00
bujinyuan
cdaa39d686 fix(meeting): Task 16 修复 code-reviewer 审计 P0 四项 + 落盘审查报告 
code-reviewer 子代理（docs/reviews/2026-04-23-phase2e-2-code-review.md）对 Phase 2e-2
整体做全栈审计，暴露 P0 × 4 / P1 × 8 / P2 × 8 / Nit × 11。本次先修 P0：

- P0-1 / P0-2 媒体资源归属越权：
  新增 assertOwnsResource(roomCode, userID, kind, id) helper，用 resourceTrackKey Redis Set
  SIsMember 校验 transport/producer/consumer 归属；统一接入 OnTransportConnect /
  OnProduceStart / OnConsumeStart / OnConsumeResume / OnProducerClose 五个信令入口，
  一次封死同类横向越权漏洞（Redis 查询失败时 fail-closed，避免抖动开口子）。
  新增 service.ErrResourceNotOwned 错误常量。
- P0-3 CreateRoom Router 失败吞错：
  旧实现 mediaErr != nil 仅 logs.Warn 继续返回成功 → zombie 房间 + 入会必败 +
  一人一会议名额被占。改为 fail-closed 补偿：LeaveRoom + MarkEnded(system_error) +
  返回新 service.ErrMediaServiceUnavailable，controller.handleError 映射为 500。
- P0-4 会议密码明文走 URL：
  join.vue onNext 旧逻辑把密码拼 ?password=xxx，浏览器历史 / DevTools 留痕。
  新增 meetingStore.draftJoinPayload 内存态草稿，join.vue 只跳转
  ?mode=join&code=xxx，preview.vue onLoad 读取后立即 draftJoinPayload = null 清空，
  邀请链接里的 password 参数一律忽略。

验证：
- cd backend/go-service && go build ./... 通过
- cd frontend && npm run build:h5 通过
- 仅剩 1 个预先存在 unusedparams warning，与本次改动无关

P1 / P2 / Nit 清单保留在审查报告，按顺序继续处理。

Made-with: Cursor
 2026-04-23 16:34:13 +08:00
bujinyuan
790996feaf feat(meeting): Task 15 UI 打磨 + 主持人四件套 + 媒体层稳定性回归修复 
- Task 15 UI：6 项原创特色（说话者流光 / 柔性网格 / 自视频浮窗 / 静音氛围色 / 入会滑入 / NetworkBadge 3 条波浪）+ 说话者双源探测（RTP audioLevel + WebAudio RMS）+ 主持人四件套（静音/开麦/转让/踢出）
- 新增 SelfVideoFloat 浮窗组件 + 4 份 design-system 页面文档（home / preview / room / invite）
- 媒体回归补丁（手工联调触发）：
  * 后端 signal_service 在 OnRoomJoin 追加 pushExistingRoomState → 向新加入者补发历史 producers + 历史成员 audio/video 状态
  * 新增 Redis Hash memberStateKey 持久化成员 audio/video enabled，OnMemberStateChanged 落盘、cleanupUserResources 清理
  * 前端 _broadcastSelfState 在本地音视频开关末尾同步 state.changed；_afterJoined 先 getRoom 再 room.join 修复后入者成员状态时序；_onMemberStateChanged 占位兜底
  * _cleanupRemoteProducer 改为按 producerId 精准清理（修复 "关音频误关视频"）
  * _onRoomEnded + createAndEnter/joinAndEnter 强化 reset（修复重入会看不到自己画面）
  * mediasoup-client ensureSend/RecvTransport 引入 in-flight Promise 锁防并发重复建连
- 文档同步：CURRENT_STATUS.md / Task 15 plan / phase2e-2 implementation plan

Made-with: Cursor
 2026-04-23 16:04:46 +08:00
bujinyuan
1222bf551d feat(phase2e-2): 会议内聊天面板落地（Task 12） 
- 新增 ChatPanel 组件：右侧抽屉式聊天 UI，支持发送/加载更早消息/未读清零
- store/meeting.js：扩展 chatHasMore/chatUnreadCount/chatHistoryLoaded 等状态与 loadMoreChats/markChatsRead 动作；WS 消息到达时按 id 去重
- room.vue：替换临时 toast 占位为 ChatPanel，绑定未读徽标、成员展示名映射；watch unread 自动在面板打开时清零
- 后端：SendChat WS 广播与 REST 响应补充 user_name/user_avatar，聊天面板无需额外查询即可展示昵称

Made-with: Cursor
 2026-04-22 15:16:17 +08:00
bujinyuan
f97fec24a8 feat(phase2e-2): 前端 mediasoup-client 集成 + Pinia meeting Store（Task 9） 
- frontend/src/api/meeting.js：12 个 REST 接口封装，统一 unwrap envelope.data
- frontend/src/services/websocket.js：新增 sendWithAck（Promise 化 + 超时 + 序列号）
- frontend/src/utils/mediasoup-client.js：MediaEngine 包装 Device/Transport/Producer/Consumer
- frontend/src/store/meeting.js：Pinia 会议状态机，桥接 14 个 WS 事件 + cleanupStaleMeetings
- frontend/src/constants/meeting.js：状态枚举 + 事件名集中管理
- frontend/src/pages/meeting/debug.vue：临时调试页（H5 原生 video/audio DOM 绕过 uni 组件限制）
- backend：meeting.consume.resume WS 事件 + create/join 响应透传 router_id + rtp_capabilities
- 文档：frontend/meeting.md、websocket.md、CURRENT_STATUS、plan 全部同步 Task 9 落地

Made-with: Cursor
 2026-04-22 11:22:49 +08:00
bujinyuan
e235e001b3 feat(phase2e-2): Go meeting 模块 12 个 REST 接口业务逻辑全量落地（Task 5） 
主要产出：
- DTO 层：app/dto/meeting_dto.go 完整定义 13 个 DTO（请求/响应/基础共用）
- Service 层：MeetingService 12 业务方法 + 11 个 sentinel 错误 + 4 辅助
  - CreateRoom/JoinRoom/LeaveRoom/EndRoom 核心生命周期
  - KickMember/TransferHost/GetRoom/ListMyMeetings 会议管理
  - InviteUsers（+ NotifyPusher）/ RedeemInviteToken 邀请链路
  - SendChat/ListChats 会议内聊天
- Controller 层：12 handler + handleError 领域错误 → HTTP 映射
- 工具层：pkg/utils/meeting_code.go（XXX-XXX-XXX + invite token）
- Stub 接口：MediaOrchestrator（Task 7 替换）+ NoopMediaOrchestrator
- 路径修正：/rooms → /rooms/mine、/invites → /invite-tokens 对齐设计
- DAO 契约修复：GetByID/GetByCode/GetByRoomAndUser/FindActiveByUser
  将 gorm.ErrRecordNotFound 转为 (nil, nil)，service 统一 nil 判定
- 安全强化：密码 bcrypt + 5 次错误锁 10 分钟；邀请 token 仅通过
  NotifyPusher.Extra 定向下发，响应不回传
- host 自动转让：host 离会时将最早加入者提升为 host
- 单点参会：用户同一时间仅能在一个活跃会议
- API 文档：docs/api/frontend/meeting.md 重写为 12 接口完整规范
- Wire 依赖注入：MediaOrchestrator + NoopMediaOrchestrator provider

验证：
- go build / go vet / wire 零告警
- 端到端 3 用户场景：12 happy path + 5 错误路径 PASS=19 / FAIL=0
  覆盖密码错/房间不存在/单点冲突/越权/邀请失效

文档同步：
- docs/progress/CURRENT_STATUS.md 增补 Task 5 章节
- .cursor/rules/project-context.mdc 更新阶段状态
- docs/plans/2026-04-21-phase2e-2-implementation.plan.md 标记 T5 ✅

下一步：Task 6（WS 信令协议 + BroadcastToMeeting 替换 PublishToUser 循环）

Made-with: Cursor
 2026-04-21 16:39:59 +08:00
bujinyuan
d22f61fb9b feat(phase2e-2): Go meeting 模块 DDL + Model + DAO + service/controller/router 骨架（Task 3+4） 
Task 3 - 数据库与持久化层：
- init.sql 追加 3 张表（meeting_rooms / meeting_participants / meeting_chats）+ 9 索引 + 全量 COMMENT
- 新增 phase2e2_migration.sql 增量升级脚本（IF NOT EXISTS 幂等）
- app/constants/meeting.go 统一会议常量（类型/状态/角色/结束原因/离会原因/默认配置/WS 事件 8 组）
- app/meeting/model 3 个 model + GORM tag（TIMESTAMP(0) 对齐现有表风格）
- app/meeting/dao 3 个 DAO 共 24 个方法：
  * meeting_room_dao：Create/Get/Exists/MarkStarted/MarkEnded（乐观锁）/UpdateHost/UpdateSettings/ListByHost/ListExpiredForCleanup
  * meeting_participant_dao：JoinRoom（事务内复用旧记录）/LeaveRoom（EXTRACT(EPOCH) DB 时间）/LeaveAllActive/TransferHost（事务）/FindActiveByUser（JOIN 单点校验）/列表计数更新
  * meeting_chat_dao：Create/ListByRoom（游标）/DeleteByRoomIDs/CountByRoom
- psql 真库跑通 CRUD + UNIQUE + CASCADE + 主持人转让事务 + duration=10s 精确匹配 8 场景

Task 4 - Go 侧 service/controller/router/wire 骨架：
- app/meeting/service/interfaces.go：NotifyPusher / UserInfoResolver / OnlineChecker 三接口（解耦 notify/contact/ws）
- app/meeting/service/meeting_service.go：MeetingService + 8 sentinel error + 17 个空方法（返回 ErrNotImplemented）
- app/meeting/controller/meeting_controller.go：12 个 Gin handler + responseNotImplemented（501）
- app/meeting/router.go：12 条路由挂 /api/v1/meeting/* 并套 jwtAuth（扁平化 router.go，与 group/notify 一致）
- app/meeting/provider.go：MeetingSet = wire.NewSet(DAO×3, Service, Controller)
- 全局 wire.go 挂入 MeetingSet + 3 条 wire.Bind（Notify/UserInfo/Online）；App struct/NewApp 加字段；router/router.go 挂载
- 存量修复：admin/provider.go 补齐 MessageManage{DAO,Service,Controller} 解决旧版 wire 重生成报 no provider found
- 验证：go build ./... / go vet ./... / wire ./app/provider 零错误；GIN_MODE=debug 启动打印全部 12 条路由；无 token curl 3 条代表性路由均返回 401（JWT 中间件生效）

关键风格修正（偏离草案）：
- 常量归入 app/constants/meeting.go 单文件（与 group.go / notify.go 同构），非草案 app/meeting/constants/
- TIMESTAMP(0) 取代草案 TIMESTAMPTZ 对齐所有现有表
- 冗余 idx_meeting_rooms_code 移除（UNIQUE 已自动建索引）
- router/provider 目录扁平化（app/meeting/router.go / app/meeting/provider.go）
- 延续项目 Go 侧"零 _test.go"风格，用代码审查 + psql 真库验证 + Playwright E2E 三层守护

文档同步：
- docs/progress/CURRENT_STATUS.md 新增 Task 3 / Task 4 交付段落
- .cursor/rules/project-context.mdc Phase 2e-2 进度推进至 Task 0-4 ✅
- docs/plans/2026-04-21-phase2e-2-implementation.plan.md Task 3 / Task 4 标记完成 + 实际产出 vs 计划差异

Made-with: Cursor
 2026-04-21 16:04:57 +08:00