Skip to content

Conversation

@loseminho
Copy link
Collaborator

📌 개요

스터디룸 JWT와 인증 통합

  • Controller에 잡은 하드 코딩된 userId 제거
  • JwtTokenProvider와 연동
  • 인증 실패 시의 예외 처리 작업
  • CurrentUser를 이용해서 실제 사용자 ID 추출

🔨 작업 내용

  • SecurityConfig.java -> 테스트를 위해 허용한 room 모든 정보 인증 필수로 변경 (주석처리)

  • RommController.java -> 하드코딩된 사용자 정보 삭제 후 CurrentUser에서 실제 사용자 ID 추출
    -> CurrentUser Mock 추가, Authorization 헤더 제거, 검증 로직을 eq(1L)를 사용해서 검증으로 변경

  • SpringDocConfig.java -> 코드 테스트를 위해 Swagger JWT 인증 버튼 생성,

🔗 관련 이슈

Closes #86

📝 참고 사항

스웨거에서 JWT 확인 시 --> 스웨거에서 로그인 후 발급 받은 토큰을 가장 상단에 자물쇠 버튼 클릭해서 나오는 모달창에
복사 붙여넣기 하면 사용 가능, (별도로 bearer지정 안해줘도 됩니다!!!)

✅ 체크리스트

  • 기능 동작 확인
  • 테스트 코드 작성
  • 문서/주석 추가 및 최신화

@loseminho loseminho self-assigned this Sep 26, 2025
@github-actions github-actions bot changed the title Feat: 스터디룸 JWT 인증 통합 Feat: 스터디룸 JWT 인증 통합 (#86) Sep 26, 2025
@KSH0326 KSH0326 self-requested a review September 26, 2025 07:35
@loseminho
Copy link
Collaborator Author

스터디 룸 내에 잡혀있어 테스트 실패한 Chat 부분 임시 허용으로 코드 수정 해서 재 푸시 했습니다!

@loseminho loseminho merged commit 87ef195 into dev Sep 26, 2025
2 checks passed
@github-actions github-actions bot deleted the feat/86 branch September 26, 2025 08:12
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.

5 participants