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

exception filter를common-jwt 모듈로 이동 #80

Closed
zhyunk opened this issue Apr 29, 2024 · 0 comments
Closed

exception filter를common-jwt 모듈로 이동 #80

zhyunk opened this issue Apr 29, 2024 · 0 comments
Assignees

Comments

@zhyunk
Copy link
Contributor

zhyunk commented Apr 29, 2024

  • exception filter를 jwt 모듈로 옮기기
    -> 이게 먹힐까? 예전에 advice는 공통 모듈에서만 구현했을때 공통 모듈에 의존성을 둔 모듈들은 먹히지 않았었던 기억이 있는데, 그 때 잘못 구현했었을 수도 있으니 재시도
      -> 문득 든 생각이 advice도 같은 패키지만 검열하나 싶다 패키지 설정을 해줬어야했던것일까?!
      저녁에 퇴근하고 다양한 시도를 계획
        -> 놓치고 있었던 중요한 포인트 같아서 구글링해보니 basepackage 설정 옵션이 있었다
          https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/web/bind/annotation/ControllerAdvice.html

Originally posted by @zhyunk in #78

@zhyunk zhyunk self-assigned this Apr 29, 2024
zhyunk added a commit that referenced this issue Apr 29, 2024
exception handler 위치를 user 모듈과 article 모듈에서 common-jwt 모듈로 이동

issue key : #80
zhyunk added a commit that referenced this issue Apr 29, 2024
- exception handler 위치를 user 모듈과 article 모듈에서 common-jwt 모듈로 이동

issue key : #80
@zhyunk zhyunk closed this as completed Apr 29, 2024
zhyunk added a commit that referenced this issue May 19, 2024
- 프로젝트 구조를 레이어 관점에서 도메인 관점으로 변경 
- 테스트 코드 또한 변경된 구조에 맞춰서 수정하는 김에 조금 더 공부해서 새롭게 작성

1차 리팩토링은 여기서 마침

issue : #67, #68, #69, #73, #75, #78, #80, #81, #82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

1 participant