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

[refac] 인증 비즈니스 로직 리팩토링 #253

Open
wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

Conversation

kgy1008
Copy link
Member

@kgy1008 kgy1008 commented Jan 13, 2025

Related Issue 📌

close #245

Description ✔️

  • 외부 서버와 통신하는 로직을 트랜잭션 범위에서 제외했습니다. (자세한 내용은 이슈에 첨부한 글 참고)
  • 그 외 자잘자잘한것도 함께 손 좀 봤습니다.

To Reviewers

UserFinder ~ UserInfoFinder 이런거 현재 보니까 Auth 패키지 하위에 있던데, User 패키지 하위에 있는게 맞지 않나용? 의견 듣고 옮기겠습니다. -> 옮겼습니다.

@kgy1008 kgy1008 added the api label Jan 13, 2025
@kgy1008 kgy1008 self-assigned this Jan 13, 2025
@kgy1008 kgy1008 changed the title [refac] 인증 로직 리팩토링 [refac] 인증관련 비즈니스 로직 리팩토링 Jan 13, 2025
@kgy1008 kgy1008 changed the title [refac] 인증관련 비즈니스 로직 리팩토링 [refac] 인증 비즈니스 로직 리팩토링 Jan 13, 2025
@PicturePark1101
Copy link
Contributor

엄 우선 코드말고 To Reviewers만 보았는데 트랜잭션은 데이터에 대해 지속성, 원자성과 같은 ACID를 보장해주기 위해 사용하지 않나요?? 저는 다른 사람이 간섭하지 않는 단일 데이터라도 ACID를 위해 트랜잭션 적용해주어야한다고 생각합니다..!

@PicturePark1101
Copy link
Contributor

PicturePark1101 commented Jan 15, 2025

user 도메인 패키지가 존재해서 저도 user 하위에 있는게 맞는 것 같아용
api/auth 패키지에는 디비 접근 컴포넌트보다는 시큐리티 관련 로직이 있는게 알맞다고 생각합니다
다만 userinfo의 경우에는 닉네임, 리프레시 토큰이 같이 저장하고 있긴한데 domain 패키지에서는 user와 함께 묶여있어서 함께 옮겨주는 것 어떠한지요

@kgy1008
Copy link
Member Author

kgy1008 commented Jan 15, 2025

user 도메인 패키지가 존재해서 저도 user 하위에 있는게 맞는 것 같아용 api/auth 패키지에는 디비 접근 컴포넌트보다는 시큐리티 관련 로직이 있는게 알맞다고 생각합니다 다만 userinfo의 경우에는 닉네임, 리프레시 토큰이 같이 저장하고 있긴한데 domain 패키지에서는 user와 함께 묶여있어서 함께 옮겨주는 것 어떠한지요

저 역시 옮기는게 맞다고 생각해서 옮기겠슴다!

@kgy1008 kgy1008 added the size/L label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[refac] 외부 네트워크 트랜잭션 범위에서 분리 (로그인 로직)
2 participants