Skip to content

[Fix] 사용자 리소스 소유자 검증 및 문제 등록 folderId null 처리 - #204

Merged
KiSeungMin merged 3 commits into
mainfrom
develop
May 7, 2026
Merged

KiSeungMin merged 3 commits into
mainfrom
develop

Conversation

@KiSeungMin

Copy link
Copy Markdown
Contributor

✔️ 연관 이슈

  • close #이슈번호

📝 작업 내용

최근 2개 커밋 기준 PR 작업 내용입니다.

  • 문제 등록 v2에서 folderId가 null로 전달될 때 사용자의 루트 폴더에 문제를 등록하도록 처리했습니다.
  • 폴더, 문제, 복습 노트, 이미지 삭제 등 사용자 리소스 접근 경로에 소유자 검증을 추가했습니다.
  • 관리자용 폴더 조회와 일반 사용자 폴더 조회를 분리해 사용자 API에서는 본인 소유 폴더만 접근하도록 변경했습니다.
  • 문제 조회, 폴더별 문제 조회, 문제 삭제, 문제 이미지 삭제, 복습 노트 상세 조회/수정/삭제, 폴더 생성/수정/삭제 시 다른 사용자의 리소스 접근을 차단하도록 서비스와 컨트롤러 호출 흐름을 정리했습니다.
  • 파일 업로드 이미지 삭제 시 문제 이미지와 풀이 이미지 각각의 소유자를 확인한 뒤 S3 파일 및 이미지 데이터를 삭제하도록 변경했습니다.
  • 다른 사용자의 폴더, 문제, 복습 노트, 이미지에 접근하는 경우 예외가 발생하는지 검증하는 테스트를 추가했습니다.
  • folderId 없이 문제 등록 v2를 호출하면 루트 폴더에 등록되는지 검증하는 테스트를 추가했습니다.

스크린샷 (선택)

  • 없음

@KiSeungMin KiSeungMin self-assigned this May 7, 2026
@KiSeungMin KiSeungMin added the fix 버그, 오류 수정 label May 7, 2026
@KiSeungMin
KiSeungMin merged commit d2489c7 into main May 7, 2026
2 checks passed
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.

1 participant