Skip to content

#559 [Fix] 클래스 의견·댓글 신고 API 클래스 권한 검사 - #560

Merged
si-zero merged 1 commit into
devfrom
fix/#559
Oct 21, 2026
Merged

si-zero merged 1 commit into
devfrom
fix/#559

Conversation

@si-zero

@si-zero si-zero commented Oct 21, 2026

Copy link
Copy Markdown
Member

#️⃣ 연관된 이슈

📝 작업 내용

🐛 Fix

내용 파일
의견·댓글 신고: 클래스 의견·댓글이면 클래스 읽기 권한 검사 (운영자·참여 중인 멤버, 아니면 403 CLASS_403_MEMBER) ReportService.java
댓글 신고: 경로의 perspectiveId 와 댓글의 관점이 다르면 404 COMMENT_NOT_FOUND ReportService.java, ReportController.java
Swagger 설명 수정: 클래스 권한 에러 추가, 자동 숨김 기준을 실제 값(10회)으로 정정 ReportController.java
클래스 API 명세서의 권한 표에 신고 추가 docs/api-specs/class-api.md

📌 공유 사항

  1. 신고는 쓰기 권한이 아니라 읽기 권한으로 판정합니다. 볼 수 있는 글이면 마감 후에도, 참여하지 않는 운영자도 신고할 수 있습니다. 부적절한 글을 마감 뒤에도 신고할 수 있어야 하고, 숨김 처리된 글은 리포트 근거 목록에서 이미 빠지도록 되어 있습니다.
  2. 일반 배틀 의견·댓글의 신고는 동작이 바뀌지 않습니다. 경로의 관점과 댓글이 다른 경우만 새로 404 가 됩니다.
  3. 테스트: 비멤버·경로 불일치·참여하지 않는 운영자·마감 후 시나리오 통합 테스트 1건 추가, 전체 575건 통과.

✅ 체크리스트

  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?

@si-zero si-zero added the 🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다. label Oct 21, 2026
@si-zero si-zero self-assigned this Oct 21, 2026
@si-zero si-zero linked an issue Oct 21, 2026 that may be closed by this pull request
4 tasks
@si-zero
si-zero merged commit 50ec87b into dev Oct 21, 2026
1 check passed
@si-zero
si-zero deleted the fix/#559 branch October 21, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Fix (버그 수정) 버그를 수정할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 Fix: 클래스 의견·댓글 신고 API 클래스 권한 검사

1 participant