Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] 비회원 게시글 조회 #200

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Conversation

dona722
Copy link
Collaborator

@dona722 dona722 commented Feb 3, 2025

🚀 Pull Request

[Fix] 비회원 게시글 조회

#️⃣ 연관된 이슈

#195

📋 작업 내용

  • 비회원, 게스트 게시글 상세 조회 가능

🔧 변경된 코드 설명

  1. PostDetailController:

    • @AuthenticationPrincipal의 인증 정보가 없을 경우(비회원)에도 NPE를 방지하기 위해 로직 추가.
    • 사용자 role(GENERAL, GUEST)을 가져오는 로직을 추가하여 사용자 유형 구분 가능.
    • memberIdrole 값을 PostDetailService에 전달하여 사용자 유형에 따른 처리 분리.
    • 비회원 및 게스트도 게시글 조회가 가능하도록 변경.
  2. PostDetailService:

    • memberIdrole을 기반으로 사용자 유형 분리:

      • 비회원, 게스트: 공개 게시글만 조회 가능. 비공개 글 접근 불가.
      • 일반 회원: 비공개 글도 본인만 조회 가능하고, 작성자 여부 확인 지원 (isMyPost).
    • isMyPost 필드 로직 수정: 일반 회원(GENERAL)만 작성자 여부를 판별하도록 변경.

    • 게시글 삭제 여부(isDeleted)를 체크하여 삭제된 게시글 접근 제한.

✅ 테스트 여부

  • 테스트 코드 실행 여부
  • 서버 실행 여부
  • 스웨거 테스트 여부

👽 비고

기타 알림 사항이 있으면 적어주세요.

@dona722 dona722 linked an issue Feb 3, 2025 that may be closed by this pull request
@dona722 dona722 merged commit ac841e1 into dev Feb 4, 2025
2 checks passed
@dona722 dona722 deleted the feat/post-detail-read-by-role-#195 branch February 7, 2025 07:17
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.

[Feat] 게시글 상세조회 권한 별 로직
3 participants