pricelees/roomescape-refactored
1
0
Fork 0
generated from pricelees/issue-pr-template
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

[#3] 인증 / 회원 코드 코틀린 마이그레이션 #4

Merged
pricelees merged 37 commits from refactor/#3 into main 2025-07-13 12:18:50 +00:00
Conversation 0 Commits 37 Files Changed 65 +1127 -1292
pricelees commented 2025-07-13 12:18:39 +00:00
Owner
Copy Link

📝 관련 이슈 및 PR

PR과 관련된 이슈 번호

✨ 작업 내용

0. 공통

  • 패키지 구조 수정(web, business, infrastructure 구조)
  • Swagger-UI 어노테이션을 별도의 인터페이스를 만들어 컨트롤러에서 분리
  • 결합도 높은 클래스 통합(ex: Member 엔티티와 Role enum 등)

1. 회원 도메인

  • 기능 자체가 적어서 변화된 내용이 크게 없음. 패키지 구조 수정과 클래스 통합 정도의 과정이 대부분이었음.

2. 인증 도메인

  • 전체적으로 코드 중복이 많아, 확장함수 및 클래스 통합으로 중복 코드를 상당히 많이 제거하였음.
  • JwtHandler와 Interceptor에서 모두 이뤄지던 null 예외 처리를 JwtHandler에서만 처리하도록 수정

🧪 테스트

  • 모든 테스트는 Kotest 기반으로 수정 & 로그인 및 토큰 처리가 필요한 API 테스트는 mocking을 활용하도록 수정
  • 향후 테스트도 꼭 필요하다고 느껴지는 테스트가 아니라면 DB 사용보다는 mocking을 활용할 예정

📚 참고 자료 및 기타

<!-- 제목 양식 --> <!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) --> ## 📝 관련 이슈 및 PR **PR과 관련된 이슈 번호** - #3 ## ✨ 작업 내용 <!-- 어떤 작업을 했는지 알려주세요! --> ### 0. 공통 - 패키지 구조 수정(web, business, infrastructure 구조) - Swagger-UI 어노테이션을 별도의 인터페이스를 만들어 컨트롤러에서 분리 - 결합도 높은 클래스 통합(ex: Member 엔티티와 Role enum 등) ### 1. 회원 도메인 - 기능 자체가 적어서 변화된 내용이 크게 없음. 패키지 구조 수정과 클래스 통합 정도의 과정이 대부분이었음. ### 2. 인증 도메인 - 전체적으로 코드 중복이 많아, 확장함수 및 클래스 통합으로 중복 코드를 상당히 많이 제거하였음. - JwtHandler와 Interceptor에서 모두 이뤄지던 null 예외 처리를 JwtHandler에서만 처리하도록 수정 ## 🧪 테스트 <!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! --> - 모든 테스트는 Kotest 기반으로 수정 & 로그인 및 토큰 처리가 필요한 API 테스트는 mocking을 활용하도록 수정 - 향후 테스트도 꼭 필요하다고 느껴지는 테스트가 아니라면 DB 사용보다는 mocking을 활용할 예정 ## 📚 참고 자료 및 기타 <!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
pricelees added the
Kind/Enhancement
 label 2025-07-13 12:18:39 +00:00
pricelees self-assigned this 2025-07-13 12:18:39 +00:00
pricelees added 37 commits 2025-07-13 12:18:40 +00:00 
- web 관련(controller, 어노테이션, interceptor, argument_resolver)
- business 관련(service..)
- infrastructure 관련(jwt..)
- API Docs 관련 정보를 담고 있는 AuthAPI 추가 및 구현
- CookieUtils를 사용하여 불필요한 컨트롤러 로직 제거
- getMemberIdFromToken의 파라미터를 nullable로 지정하였음. null인 경우 parseClaimJws에서 예외가 발생하기 때문
pricelees merged commit 22e6ad4f71 into main 2025-07-13 12:18:50 +00:00
pricelees deleted branch refactor/#3 2025-07-13 12:18:50 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Kind/Enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
pricelees
No Assignees pricelees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pricelees/roomescape-refactored#4
No description provided.
Delete Branch "refactor/#3"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?