Skip to content

Conversation

@MEOHIN
Copy link
Collaborator

@MEOHIN MEOHIN commented Oct 2, 2025

📢 기능 설명

댓글 수정/삭제 시 500 오류가 발생하던 문제를 해결했습니다.

연결된 issue

close #222



🩷 Approve 하기 전 확인해주세요!

  • CommentService
    • 댓글 수정 및 삭제 로직에서 현재 로그인한 사용자와 댓글 작성자를 비교할 때, 기존에 객체(comment.getUser().equals(user))를 비교하던 방식에서 ID 값(comment.getUser().getId().equals(user.getId()))을 비교하는 방식으로 변경했습니다.
  • CustomAuthenticationFilter
    1. Authorization Header 우선: Authorization: Bearer <token> 헤더에서 토큰을 먼저 확인하도록 변경했습니다.
    2. Cookie Fallback 유지: 헤더에 토큰이 없는 경우 기존처럼 쿠키에서 accessToken을 가져와 하위 호환성을 유지합니다.
    3. 토큰 재발급 로직 분리: 만료된 토큰 재발급 및 쿠키 설정은 토큰을 쿠키에서 가져온 경우에만 실행되도록 조건을 추가했습니다. (헤더 기반 인증 시 불필요한 쿠키 재설정 방지)
    4. 로그 강화: 인증 필터의 흐름을 추적할 수 있도록 상세 debug 로그를 추가했습니다.

✅ 체크리스트

  • PR 제목 규칙 잘 지켰는가?
  • 추가/수정사항을 설명하였는가?
  • 이슈넘버를 적었는가?
  • Approve 하기 전 확인 사항 체크했는가?

MEOHIN added 4 commits October 2, 2025 10:40
댓글 수정 및 삭제 로직에서 현재 로그인한 사용자와 댓글 작성자를 비교할 때,
기존에 객체(`comment.getUser().equals(user)`)를 비교하던 방식에서 ID 값(`comment.getUser().getId().equals(user.getId())`)을 비교하는 방식으로 변경
@MEOHIN MEOHIN self-assigned this Oct 2, 2025
@MEOHIN MEOHIN added the fix 버그 수정 label Oct 2, 2025
@MEOHIN MEOHIN merged commit 56fe1f3 into dev Oct 2, 2025
1 check passed
@MEOHIN MEOHIN deleted the feat#222 branch October 2, 2025 01:46
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] 댓글 수정/삭제 500 오류 해결

3 participants