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

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

seokjin8678
Copy link
Collaborator

📌 관련 이슈

✨ PR 세부 내용

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

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

image

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

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

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

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

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 ⚙️ 리팩터링 리팩터링에 관련된 작업 labels Jun 7, 2024
@seokjin8678 seokjin8678 self-assigned this Jun 7, 2024
@seokjin8678 seokjin8678 linked an issue Jun 7, 2024 that may be closed by this pull request
@github-actions github-actions bot requested review from BGuga, carsago and xxeol2 June 7, 2024 09:37
Copy link

github-actions bot commented Jun 7, 2024

Test Results

244 files  244 suites   27s ⏱️
792 tests 792 ✅ 0 💤 0 ❌
810 runs  810 ✅ 0 💤 0 ❌

Results for commit 8dc1dce.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드에 관련된 작업 ⚙️ 리팩터링 리팩터링에 관련된 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE] auth 패키지의 구조를 정리한다.
1 participant