search

woowacourse-teams / 2023-festa-go

🎪 페스타고, 대학 축제를 더욱 즐겁게!
71 stars 8 forks source link

[BE] refactor: auth 패키지 내 클래스 정리 (#1005) #1006

Open seokjin8678 opened 5 months ago

seokjin8678 commented 5 months ago

📌 관련 이슈

✨ PR 세부 내용

auth 패키지 내 제자리를 찾아가지 못하고 있는 클래스들을 정리했습니다.

다만 DSM을 확인하니 양방향 의존이 있는 곳이 있길래 제거해보려 했으나...

image

common 패키지에서 GlobalExceptionHandler 클래스가 auth 패키지의 AuthenticateContext과 Role을 의존하고 있더군요. 😂

양방향 의존을 제거하려면 AuthenticateContext를 인터페이스로 만든 뒤,AuthenticateContextImpl을 만들고, Role과 관련된 어노테이션들을 모두 common 패키지에 이동시켜야 합니다.

하지만 양방향 의존을 없애려고 인증에 관련된 여러 클래스들을 아무 상관이 없는 common에 두는 것이 오히려 역효과만 발생할 것 같습니다. (직관성에 어긋남)

따라서 우선 양방향 의존이 발생하게 두되, 거슬린다면 아예 common 패키지에 있는 GlobalExceptionHandler 클래스를 다른 패키지로 이동시켜서 해결을 해봐야 할 것 같네요. (api 모듈)

github-actions[bot] commented 5 months ago

Test Results

244 files  244 suites   27s :stopwatch: 792 tests 792 :white_check_mark: 0 :zzz: 0 :x: 810 runs  810 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 8dc1dce7.