Merge pull request #85 from YAPP-Github/refactor/#83-auto-login

wjdrjs00 · web-flow · commit a93e2d3560d2 · 2025-08-09T20:51:09.000+09:00
[Refactor/#83] 자동 로그인 로직 변경
diff --git a/app/src/main/java/com/threegap/bitnagil/MainNavHost.kt b/app/src/main/java/com/threegap/bitnagil/MainNavHost.kt
@@ -38,6 +38,16 @@ fun MainNavHost(
                         popUpTo<Route.Splash> { inclusive = true }
                     }
                 },
+                navigateToTermsAgreement = {
+                    navigator.navController.navigate(Route.TermsAgreement) {
+                        popUpTo<Route.Splash> { inclusive = true }
+                    }
+                },
+                navigateToOnboarding = {
+                    navigator.navController.navigate(Route.OnBoarding()) {
+                        popUpTo<Route.Splash> { inclusive = true }
+                    }
+                },
                 navigateToHome = navigator::navigateToHomeAndClearStack,
             )
         }
diff --git a/core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStore.kt b/core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStore.kt
@@ -5,14 +5,8 @@ import kotlinx.coroutines.flow.Flow
 
 interface AuthTokenDataStore {
     val tokenFlow: Flow<AuthToken>
-
-    suspend fun hasToken(): Boolean
-
     suspend fun updateAuthToken(accessToken: String, refreshToken: String)
-
     suspend fun updateAccessToken(accessToken: String)
-
     suspend fun updateRefreshToken(refreshToken: String)
-
     suspend fun clearAuthToken()
 }
diff --git a/core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImpl.kt b/core/datastore/src/main/java/com/threegap/bitnagil/datastore/auth/storage/AuthTokenDataStoreImpl.kt
@@ -3,24 +3,12 @@ package com.threegap.bitnagil.datastore.auth.storage
 import androidx.datastore.core.DataStore
 import com.threegap.bitnagil.datastore.auth.model.AuthToken
 import kotlinx.coroutines.flow.Flow
-import kotlinx.coroutines.flow.firstOrNull
 
 class AuthTokenDataStoreImpl(
     private val dataStore: DataStore<AuthToken>,
 ) : AuthTokenDataStore {
     override val tokenFlow: Flow<AuthToken> = dataStore.data
 
-    override suspend fun hasToken(): Boolean {
-        return try {
-            val currentToken = dataStore.data.firstOrNull()
-            currentToken?.let {
-                !it.accessToken.isNullOrEmpty() && !it.refreshToken.isNullOrEmpty()
-            } ?: false
-        } catch (e: Exception) {
-            false
-        }
-    }
-
     override suspend fun updateAuthToken(accessToken: String, refreshToken: String) {
         try {
             dataStore.updateData {
@@ -58,8 +46,4 @@ class AuthTokenDataStoreImpl(
             throw e
         }
     }
-
-    companion object {
-        private const val TAG = "AuthTokenDataStore"
-    }
 }
diff --git a/core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt b/core/network/src/main/java/com/threegap/bitnagil/network/auth/TokenAuthenticator.kt
@@ -19,6 +19,7 @@ class TokenAuthenticator(
     private val authMutex = Mutex()
 
     override fun authenticate(route: Route?, response: Response): Request? {
+        if (response.request.header(AUTO_LOGIN_HEADER) != null) return null
         if (!shouldRetry(response)) return null
 
         val currentToken = runBlocking { tokenProvider.getAccessToken() }
@@ -82,6 +83,7 @@ class TokenAuthenticator(
     private fun buildRequestWithToken(originalRequest: Request, token: String): Request {
         return originalRequest.newBuilder()
             .header(AUTHORIZATION, "$TOKEN_PREFIX $token")
+            .removeHeader(AUTO_LOGIN_HEADER)
             .build()
     }
 
@@ -96,5 +98,6 @@ class TokenAuthenticator(
         private const val AUTHORIZATION = "Authorization"
         private const val TOKEN_PREFIX = "Bearer"
         private const val SUCCESS_CODE = "CO000"
+        private const val AUTO_LOGIN_HEADER = "Auto-Login"
     }
 }
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/datasource/AuthLocalDataSource.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/datasource/AuthLocalDataSource.kt
@@ -1,9 +1,7 @@
 package com.threegap.bitnagil.data.auth.datasource
 
 interface AuthLocalDataSource {
-    suspend fun hasToken(): Boolean
-
+    suspend fun getRefreshToken(): String?
     suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit>
-
     suspend fun clearAuthToken(): Result<Unit>
 }
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/datasource/AuthRemoteDataSource.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/datasource/AuthRemoteDataSource.kt
@@ -6,10 +6,8 @@ import com.threegap.bitnagil.data.auth.model.response.LoginResponseDto
 
 interface AuthRemoteDataSource {
     suspend fun login(socialAccessToken: String, loginRequestDto: LoginRequestDto): Result<LoginResponseDto>
-
     suspend fun submitAgreement(termsAgreementRequestDto: TermsAgreementRequestDto): Result<Unit>
-
     suspend fun logout(): Result<Unit>
-
     suspend fun withdrawal(): Result<Unit>
+    suspend fun reissueToken(refreshToken: String): Result<LoginResponseDto>
 }
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/datasourceimpl/AuthLocalDataSourceImpl.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/datasourceimpl/AuthLocalDataSourceImpl.kt
@@ -2,13 +2,17 @@ package com.threegap.bitnagil.data.auth.datasourceimpl
 
 import com.threegap.bitnagil.data.auth.datasource.AuthLocalDataSource
 import com.threegap.bitnagil.datastore.auth.storage.AuthTokenDataStore
+import kotlinx.coroutines.flow.firstOrNull
 import javax.inject.Inject
 
 class AuthLocalDataSourceImpl @Inject constructor(
     private val authTokenDataStore: AuthTokenDataStore,
 ) : AuthLocalDataSource {
 
-    override suspend fun hasToken(): Boolean = authTokenDataStore.hasToken()
+    override suspend fun getRefreshToken(): String? =
+        runCatching {
+            authTokenDataStore.tokenFlow.firstOrNull()?.refreshToken
+        }.getOrNull()
 
     override suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit> =
         runCatching {
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/datasourceimpl/AuthRemoteDataSourceImpl.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/datasourceimpl/AuthRemoteDataSourceImpl.kt
@@ -31,4 +31,9 @@ class AuthRemoteDataSourceImpl @Inject constructor(
         safeUnitApiCall {
             authService.postWithdrawal()
         }
+
+    override suspend fun reissueToken(refreshToken: String): Result<LoginResponseDto> =
+        safeApiCall {
+            authService.postReissueToken(refreshToken)
+        }
 }
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/repositoryimpl/AuthRepositoryImpl.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/repositoryimpl/AuthRepositoryImpl.kt
@@ -18,11 +18,6 @@ class AuthRepositoryImpl @Inject constructor(
         authRemoteDataSource.login(socialAccessToken, LoginRequestDto(socialType))
             .map { it.toDomain() }
 
-    override suspend fun hasToken(): Boolean = authLocalDataSource.hasToken()
-
-    override suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit> =
-        authLocalDataSource.updateAuthToken(accessToken, refreshToken)
-
     override suspend fun submitAgreement(termsAgreement: TermsAgreement): Result<Unit> =
         authRemoteDataSource.submitAgreement(
             termsAgreement.toDto(),
@@ -39,4 +34,16 @@ class AuthRepositoryImpl @Inject constructor(
             if (it.isSuccess) authLocalDataSource.clearAuthToken()
         }
     }
+
+    override suspend fun reissueToken(refreshToken: String): Result<AuthSession> =
+        authRemoteDataSource.reissueToken(refreshToken).map { it.toDomain() }
+
+    override suspend fun getRefreshToken(): String? =
+        authLocalDataSource.getRefreshToken()
+
+    override suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit> =
+        authLocalDataSource.updateAuthToken(accessToken, refreshToken)
+
+    override suspend fun clearAuthToken(): Result<Unit> =
+        authLocalDataSource.clearAuthToken()
 }
diff --git a/data/src/main/java/com/threegap/bitnagil/data/auth/service/AuthService.kt b/data/src/main/java/com/threegap/bitnagil/data/auth/service/AuthService.kt
@@ -27,4 +27,10 @@ interface AuthService {
 
     @POST("/api/v1/auth/logout")
     suspend fun postLogout(): BaseResponse<Unit>
+
+    @POST("/api/v1/auth/token/reissue")
+    @Headers("No-Service-Token: true", "Auto-Login: true")
+    suspend fun postReissueToken(
+        @Header("Refresh-Token") refreshToken: String,
+    ): BaseResponse<LoginResponseDto>
 }
diff --git a/domain/src/main/java/com/threegap/bitnagil/domain/auth/model/UserRole.kt b/domain/src/main/java/com/threegap/bitnagil/domain/auth/model/UserRole.kt
@@ -3,6 +3,9 @@ package com.threegap.bitnagil.domain.auth.model
 enum class UserRole {
     USER,
     GUEST,
+    ONBOARDING,
+    WITHDRAWN,
+    UNKNOWN,
     ;
 
     fun isGuest() = this == GUEST
@@ -12,7 +15,9 @@ enum class UserRole {
             when (value) {
                 "USER" -> USER
                 "GUEST" -> GUEST
-                else -> throw IllegalArgumentException("Unknown role: $value")
+                "ONBOARDING" -> ONBOARDING
+                "WITHDRAWN" -> WITHDRAWN
+                else -> UNKNOWN
             }
     }
 }
diff --git a/domain/src/main/java/com/threegap/bitnagil/domain/auth/repository/AuthRepository.kt b/domain/src/main/java/com/threegap/bitnagil/domain/auth/repository/AuthRepository.kt
@@ -5,14 +5,12 @@ import com.threegap.bitnagil.domain.auth.model.TermsAgreement
 
 interface AuthRepository {
     suspend fun login(socialAccessToken: String, socialType: String): Result<AuthSession>
-
-    suspend fun hasToken(): Boolean
-
-    suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit>
-
     suspend fun submitAgreement(termsAgreement: TermsAgreement): Result<Unit>
-
     suspend fun logout(): Result<Unit>
-
     suspend fun withdrawal(): Result<Unit>
+
+    suspend fun reissueToken(refreshToken: String): Result<AuthSession>
+    suspend fun getRefreshToken(): String?
+    suspend fun updateAuthToken(accessToken: String, refreshToken: String): Result<Unit>
+    suspend fun clearAuthToken(): Result<Unit>
 }
diff --git a/domain/src/main/java/com/threegap/bitnagil/domain/auth/usecase/AutoLoginUseCase.kt b/domain/src/main/java/com/threegap/bitnagil/domain/auth/usecase/AutoLoginUseCase.kt
@@ -0,0 +1,29 @@
+package com.threegap.bitnagil.domain.auth.usecase
+
+import com.threegap.bitnagil.domain.auth.model.UserRole
+import com.threegap.bitnagil.domain.auth.repository.AuthRepository
+import javax.inject.Inject
+
+class AutoLoginUseCase @Inject constructor(
+    private val authRepository: AuthRepository,
+) {
+    suspend operator fun invoke(): UserRole {
+        val refreshToken = authRepository.getRefreshToken()
+        if (refreshToken.isNullOrEmpty()) return UserRole.UNKNOWN
+
+        return authRepository.reissueToken(refreshToken)
+            .onSuccess { authSession ->
+                authRepository.updateAuthToken(
+                    accessToken = authSession.accessToken,
+                    refreshToken = authSession.refreshToken,
+                )
+            }
+            .onFailure {
+                authRepository.clearAuthToken()
+            }
+            .fold(
+                onSuccess = { authSession -> authSession.role },
+                onFailure = { UserRole.UNKNOWN },
+            )
+    }
+}
diff --git a/domain/src/main/java/com/threegap/bitnagil/domain/auth/usecase/HasTokenUseCase.kt b/domain/src/main/java/com/threegap/bitnagil/domain/auth/usecase/HasTokenUseCase.kt
diff --git a/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/SplashScreen.kt b/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/SplashScreen.kt
@@ -29,12 +29,16 @@ import org.orbitmvi.orbit.compose.collectSideEffect
 @Composable
 fun SplashScreenContainer(
     navigateToIntro: () -> Unit,
+    navigateToTermsAgreement: () -> Unit,
+    navigateToOnboarding: () -> Unit,
     navigateToHome: () -> Unit,
     viewModel: SplashViewModel = hiltViewModel(),
 ) {
     viewModel.collectSideEffect { sideEffect ->
         when (sideEffect) {
             is SplashSideEffect.NavigateToIntro -> navigateToIntro()
+            is SplashSideEffect.NavigateToTermsAgreement -> navigateToTermsAgreement()
+            is SplashSideEffect.NavigateToOnboarding -> navigateToOnboarding()
             is SplashSideEffect.NavigateToHome -> navigateToHome()
         }
     }
diff --git a/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/SplashViewModel.kt b/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/SplashViewModel.kt
@@ -2,39 +2,42 @@ package com.threegap.bitnagil.presentation.splash
 
 import androidx.lifecycle.SavedStateHandle
 import androidx.lifecycle.viewModelScope
-import com.threegap.bitnagil.domain.auth.usecase.HasTokenUseCase
+import com.threegap.bitnagil.domain.auth.model.UserRole
+import com.threegap.bitnagil.domain.auth.usecase.AutoLoginUseCase
 import com.threegap.bitnagil.presentation.common.mviviewmodel.MviViewModel
 import com.threegap.bitnagil.presentation.splash.model.SplashIntent
 import com.threegap.bitnagil.presentation.splash.model.SplashSideEffect
 import com.threegap.bitnagil.presentation.splash.model.SplashState
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
+import kotlinx.coroutines.withTimeoutOrNull
 import org.orbitmvi.orbit.syntax.simple.SimpleSyntax
 import javax.inject.Inject
 
 @HiltViewModel
 class SplashViewModel @Inject constructor(
-    private val savedStateHandle: SavedStateHandle,
-    private val hasTokenUseCase: HasTokenUseCase,
+    savedStateHandle: SavedStateHandle,
+    private val autoLoginUseCase: AutoLoginUseCase,
 ) : MviViewModel<SplashState, SplashSideEffect, SplashIntent>(
     initState = SplashState(),
     savedStateHandle = savedStateHandle,
 ) {
 
-    private var hasToken: Boolean? = null
-
     init {
-        checkTokenStatus()
+        performAutoLogin()
     }
 
     override suspend fun SimpleSyntax<SplashState, SplashSideEffect>.reduceState(
         intent: SplashIntent,
         state: SplashState,
     ): SplashState? =
         when (intent) {
-            is SplashIntent.SetTokenChecked -> {
-                state.copy(isTokenChecked = intent.hasToken != null)
+            is SplashIntent.SetUserRole -> {
+                state.copy(
+                    userRole = intent.userRole,
+                    isAutoLoginCompleted = true,
+                )
             }
 
             is SplashIntent.NavigateToIntro -> {
@@ -46,34 +49,46 @@ class SplashViewModel @Inject constructor(
                 sendSideEffect(SplashSideEffect.NavigateToHome)
                 null
             }
+
+            is SplashIntent.NavigateToTermsAgreement -> {
+                sendSideEffect(SplashSideEffect.NavigateToTermsAgreement)
+                null
+            }
+
+            is SplashIntent.NavigateToOnboarding -> {
+                sendSideEffect(SplashSideEffect.NavigateToOnboarding)
+                null
+            }
         }
 
-    private fun checkTokenStatus() {
+    private fun performAutoLogin() {
         viewModelScope.launch {
             try {
-                hasToken = hasTokenUseCase()
-                sendIntent(SplashIntent.SetTokenChecked(hasToken))
+                val userRole = withTimeoutOrNull(5000) {
+                    autoLoginUseCase()
+                }
+                sendIntent(SplashIntent.SetUserRole(userRole))
             } catch (e: Exception) {
-                hasToken = false
-                sendIntent(SplashIntent.SetTokenChecked(false))
+                sendIntent(SplashIntent.SetUserRole(null))
             }
         }
     }
 
     fun onAnimationCompleted() {
-        val tokenResult = hasToken
-        if (tokenResult == null) {
+        val splashState = container.stateFlow.value
+        if (!splashState.isAutoLoginCompleted) {
             viewModelScope.launch {
                 delay(100)
                 onAnimationCompleted()
             }
             return
         }
 
-        if (tokenResult) {
-            sendIntent(SplashIntent.NavigateToHome)
-        } else {
-            sendIntent(SplashIntent.NavigateToIntro)
+        when (splashState.userRole) {
+            UserRole.GUEST -> sendIntent(SplashIntent.NavigateToTermsAgreement)
+            UserRole.USER -> sendIntent(SplashIntent.NavigateToHome)
+            UserRole.ONBOARDING -> sendIntent(SplashIntent.NavigateToOnboarding)
+            else -> sendIntent(SplashIntent.NavigateToIntro)
         }
     }
 }
diff --git a/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashIntent.kt b/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashIntent.kt
diff --git a/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashSideEffect.kt b/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashSideEffect.kt
diff --git a/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashState.kt b/presentation/src/main/java/com/threegap/bitnagil/presentation/splash/model/SplashState.kt

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,16 @@ fun MainNavHost(`
`38`	`38`	`popUpTo<Route.Splash> { inclusive = true }`
`39`	`39`	`}`
`40`	`40`	`},`
	`41`	`+ navigateToTermsAgreement = {`
	`42`	`+ navigator.navController.navigate(Route.TermsAgreement) {`
	`43`	`+ popUpTo<Route.Splash> { inclusive = true }`
	`44`	`+ }`
	`45`	`+ },`
	`46`	`+ navigateToOnboarding = {`
	`47`	`+ navigator.navController.navigate(Route.OnBoarding()) {`
	`48`	`+ popUpTo<Route.Splash> { inclusive = true }`
	`49`	`+ }`
	`50`	`+ },`
`41`	`51`	`navigateToHome = navigator::navigateToHomeAndClearStack,`
`42`	`52`	`)`
`43`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@ class TokenAuthenticator(`
`19`	`19`	`private val authMutex = Mutex()`
`20`	`20`
`21`	`21`	`override fun authenticate(route: Route?, response: Response): Request? {`
	`22`	`+ if (response.request.header(AUTO_LOGIN_HEADER) != null) return null`
`22`	`23`	`if (!shouldRetry(response)) return null`
`23`	`24`
`24`	`25`	`val currentToken = runBlocking { tokenProvider.getAccessToken() }`
`@@ -82,6 +83,7 @@ class TokenAuthenticator(`
`82`	`83`	`private fun buildRequestWithToken(originalRequest: Request, token: String): Request {`
`83`	`84`	`return originalRequest.newBuilder()`
`84`	`85`	`.header(AUTHORIZATION, "$TOKEN_PREFIX $token")`
	`86`	`+ .removeHeader(AUTO_LOGIN_HEADER)`
`85`	`87`	`.build()`
`86`	`88`	`}`
`87`	`89`
`@@ -96,5 +98,6 @@ class TokenAuthenticator(`
`96`	`98`	`private const val AUTHORIZATION = "Authorization"`
`97`	`99`	`private const val TOKEN_PREFIX = "Bearer"`
`98`	`100`	`private const val SUCCESS_CODE = "CO000"`
	`101`	`+ private const val AUTO_LOGIN_HEADER = "Auto-Login"`
`99`	`102`	`}`
`100`	`103`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,4 +31,9 @@ class AuthRemoteDataSourceImpl @Inject constructor(`
`31`	`31`	`safeUnitApiCall {`
`32`	`32`	`authService.postWithdrawal()`
`33`	`33`	`}`
	`34`	`+`
	`35`	`+ override suspend fun reissueToken(refreshToken: String): Result<LoginResponseDto> =`
	`36`	`+ safeApiCall {`
	`37`	`+ authService.postReissueToken(refreshToken)`
	`38`	`+ }`
`34`	`39`	`}`