Conversation
📝 WalkthroughWalkthrough채팅 API에서 Possibly related PRs
🚥 Pre-merge checks | ✅ 1 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/config/sentry.ts`:
- Around line 38-41: Sentry.replayIntegration 설정에서 maskAllText: false 및
blockAllMedia: false로 인해 사용자 채팅·미디어 등 민감한 정보가 전송될 위험이 있으니
Sentry.replayIntegration 호출(Sentry.replayIntegration)에서 민감 데이터가 캡처되지 않도록
maskAllText와 blockAllMedia를 true로 변경하거나, 의도된 경우 특정 요소만 마스킹하도록 mask 옵션을 사용해 민감한
셀렉터(예: 채팅 입력, 프로필 이미지 등)를 명시적으로 제외하도록 설정을 업데이트하세요.
ℹ️ Review info
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
src/apis/chat/entity.tssrc/apis/chat/index.tssrc/config/sentry.tssrc/pages/Chat/hooks/useChat.tssrc/pages/Guide/hooks/useGuideSlider.tssrc/pages/Guide/index.tsx
| Sentry.replayIntegration({ | ||
| maskAllText: true, | ||
| blockAllMedia: true, | ||
| maskAllText: false, | ||
| blockAllMedia: false, | ||
| }), |
There was a problem hiding this comment.
개인정보 노출 위험 확인 필요
maskAllText: false와 blockAllMedia: false 설정으로 Sentry 리플레이에서 사용자 텍스트와 미디어가 그대로 녹화됩니다. 채팅 내용 등 민감한 정보가 Sentry에 전송될 수 있어 GDPR/개인정보보호 관점에서 검토가 필요합니다.
의도된 변경이라면 특정 요소만 마스킹하는 mask 옵션 사용을 권장합니다.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/config/sentry.ts` around lines 38 - 41, Sentry.replayIntegration 설정에서
maskAllText: false 및 blockAllMedia: false로 인해 사용자 채팅·미디어 등 민감한 정보가 전송될 위험이 있으니
Sentry.replayIntegration 호출(Sentry.replayIntegration)에서 민감 데이터가 캡처되지 않도록
maskAllText와 blockAllMedia를 true로 변경하거나, 의도된 경우 특정 요소만 마스킹하도록 mask 옵션을 사용해 민감한
셀렉터(예: 채팅 입력, 프로필 이미지 등)를 명시적으로 제외하도록 설정을 업데이트하세요.
Summary by CodeRabbit
릴리스 노트
개선 사항
기타