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

fix: 로그인시 디바이스 활성화가 안되던 이슈 수정 #542

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

hectick
Copy link
Collaborator

@hectick hectick commented Nov 16, 2023

🔥 연관 이슈

클래스단에 달려있는 @Transactional(readOnly = true)가 자꾸 괴롭히는군용

카카오톡에서도 이야기를 했지만 메모를 해봅니다.
일단 login메서드에 @Transactional 어노테이션을 달아줌으로써 해결하긴 하였으나,
login 메서드에서는 이벤트를 제외하면 조회해서 체크하는 로직밖에 없기 때문에
과연 login메서드가 이벤트에서 쓰기연산이 발생할 일을 파악해서 @Transactional을 쓰도록 바꾸는 것이 맞는것인가?
하는 생각이 갑자기 들었습니다.
그렇다고 디바이스를 활성화/비활성화하는 메서드에 Require_new를 달자니 데드락이 걸리네요

결론: 이참에 파사드 패턴을 고려해보는 것이 어떨까요 ~

@hectick hectick added 🐞 bug Something isn't working ❤️ Backend for BackEnd labels Nov 16, 2023
@hectick hectick self-assigned this Nov 16, 2023
Copy link

Unit Test Results

  70 files  ±0    70 suites  ±0   24s ⏱️ -6s
369 tests +1  369 ✔️ +1  0 💤 ±0  0 ±0 
372 runs  +1  372 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit 9fa9aff. ± Comparison against base commit 70b333f.

This pull request removes 2 and adds 3 tests. Note that renamed tests count towards both.
edonymyeon.backend.auth.application.AuthServiceTest ‑ 로그인 이후 디바이스 교체 작업을 수행한다
edonymyeon.backend.auth.application.AuthServiceTest ‑ 소셜로그인 이후에도 디바이스 교체 작업을 수행한다
edonymyeon.backend.auth.application.AuthServiceTest ‑ 로그인 이후 디바이스 교체 작업을 수행한다 (DeviceRepository)
edonymyeon.backend.auth.application.AuthServiceTest ‑ 소셜로그인 이후에도 디바이스 교체 작업을 수행한다 (DeviceRepository)
edonymyeon.backend.auth.application.AuthServiceTest ‑ 최초의 소셜로그인시 사용한 디바이스 정보를 함께 저장한다 (EntityManager)

@hectick hectick merged commit 79409c5 into dev Nov 16, 2023
3 checks passed
@hectick hectick deleted the fix/#541_로그인_디바이스토큰 branch November 16, 2023 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❤️ Backend for BackEnd 🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

로그인시 디바이스 토큰이 활성화되지 않는 이슈
1 participant