Skip to content

fix(notify): 대댓글 알림 비밀 처리 및 차단 관계 수신자 오류 수정#170

Merged
coldsunn merged 1 commit into
developfrom
fix/comment-notification
May 24, 2026
Merged

fix(notify): 대댓글 알림 비밀 처리 및 차단 관계 수신자 오류 수정#170
coldsunn merged 1 commit into
developfrom
fix/comment-notification

Conversation

@coldsunn
Copy link
Copy Markdown
Collaborator

@coldsunn coldsunn commented May 24, 2026

📝 요약(Summary)

문제:

  1. 비밀 대댓글 내용이 열람 권한 없는 참여자에게 알림으로 노출
  • 공개 부모 댓글 아래 비밀 대댓글 작성 시, 일반 참여자들은 FCM 푸시 알림으로 내용이 노출됨.
  1. 참여자인 리포트 당사자가 비밀 대댓글 알림 미수신
  • 공개 부모 댓글 스레드에 리포트 당사자가 대댓글을 쓴 이후(참여자 등록), 누군가 비밀 대댓글을 달면 리포트 당사자는 열람 권한이 있음에도 COMMENT_ON_MY_REPORT(참여자라 skip) + REPLY_ON_PARTICIPATED_COMMENT(비밀이라 루프 skip) 모두 발송되지 않음.
  1. 차단 관계인 부모 댓글 작성자·참여자에게 알림 발송
  • A가 B를 차단해도 둘 다 리포트 당사자 C의 친구이면 같은 스레드에 참여 가능. B가 새 대댓글 작성 시 A(부모 댓글 작성자 또는 참여자)는 B의 대댓글을 앱에서 볼 수 없지만 알림은 수신됨.

수정:

  • SubCommentCreatedEvent에 isSecret, parentSecret 필드 추가
  • CommentCommandService: 이벤트에 finalIsSecret, parentComment.isSecret() 전달
  • CommentNotificationEventListener:
    • 부모 댓글 작성자 알림에 차단 관계 체크 추가
    • 참여자 루프: isSecret && !parentSecret이면 skip
    • 단, 리포트 당사자가 참여자인 경우 REPLY_ON_PARTICIPATED_COMMENT 별도 발송 (열람 가능)
    • 비밀 부모 대댓글은 참여자가 원래부터 권한자뿐이므로 루프 유지

🔗 Related Issue

  • Closes:

💬 공유사항

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 제목을 커밋 메시지 컨벤션에 맞게 작성했습니다.

비밀 대댓글 내용이 열람 권한 없는 참여자에게 알림으로 노출되는 문제를 해결하기 위해 SubCommentCreatedEvent에 isSecret, parentSecret 필드 추가
후 CommentNotificationEventListener에서 부모 댓글 작성자 알림에 차단 관계 체크 추가, isSecret && !parentSecret이면 skip,
리포트 당사자가 참여자인 경우는 별도 발송
@coldsunn coldsunn merged commit 635a7e6 into develop May 24, 2026
1 check passed
@coldsunn coldsunn deleted the fix/comment-notification branch May 24, 2026 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant