-
Notifications
You must be signed in to change notification settings - Fork 2
Feat: 스터디룸 JWT 인증 통합 (#86) #93
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
joyewon0705
approved these changes
Sep 26, 2025
KSH0326
approved these changes
Sep 26, 2025
Collaborator
Author
|
스터디 룸 내에 잡혀있어 테스트 실패한 Chat 부분 임시 허용으로 코드 수정 해서 재 푸시 했습니다! |
jueunk617
approved these changes
Sep 26, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 개요
스터디룸 JWT와 인증 통합
🔨 작업 내용
SecurityConfig.java -> 테스트를 위해 허용한 room 모든 정보 인증 필수로 변경 (주석처리)
RommController.java -> 하드코딩된 사용자 정보 삭제 후 CurrentUser에서 실제 사용자 ID 추출
-> CurrentUser Mock 추가, Authorization 헤더 제거, 검증 로직을 eq(1L)를 사용해서 검증으로 변경
SpringDocConfig.java -> 코드 테스트를 위해 Swagger JWT 인증 버튼 생성,
🔗 관련 이슈
Closes #86
📝 참고 사항
스웨거에서 JWT 확인 시 --> 스웨거에서 로그인 후 발급 받은 토큰을 가장 상단에 자물쇠 버튼 클릭해서 나오는 모달창에
복사 붙여넣기 하면 사용 가능, (별도로 bearer지정 안해줘도 됩니다!!!)
✅ 체크리스트