Refactor: LoginService 분리 및 네트워크 인증 로직 개선

- 인증이 필요 없는 API 처리를 위한 LoginService 추가 및 관련 로직 이동
- AuthInterceptor 내 No-Service-Token 헤더 체크 및 삭제 로직 제거

Author: wjdrjs00

app/src/main/java/com/threegap/bitnagil/di/data/ServiceModule.kt

@@ -2,6 +2,7 @@ package com.threegap.bitnagil.di.data
 
 import com.threegap.bitnagil.data.address.service.AddressService
 import com.threegap.bitnagil.data.auth.service.AuthService
+import com.threegap.bitnagil.data.auth.service.LoginService
 import com.threegap.bitnagil.data.emotion.service.EmotionService
 import com.threegap.bitnagil.data.file.service.FileService
 import com.threegap.bitnagil.data.onboarding.service.OnBoardingService
@@ -30,6 +31,11 @@ object ServiceModule {
     fun provideAuthService(@Auth retrofit: Retrofit): AuthService =
         retrofit.create(AuthService::class.java)
 
+    @Provides
+    @Singleton
+    fun provideLoginService(@NoneAuth retrofit: Retrofit): LoginService =
+        retrofit.create(LoginService::class.java)
+
     @Provides
     @Singleton
     fun providerOnBoardingService(@Auth retrofit: Retrofit): OnBoardingService =

core/network/src/main/java/com/threegap/bitnagil/network/auth/AuthInterceptor.kt

@@ -3,24 +3,16 @@ package com.threegap.bitnagil.network.auth
 import com.threegap.bitnagil.network.token.TokenProvider
 import kotlinx.coroutines.runBlocking
 import okhttp3.Interceptor
-import okhttp3.Request
 import okhttp3.Response
 
 class AuthInterceptor(
     private val tokenProvider: TokenProvider,
 ) : Interceptor {
     override fun intercept(chain: Interceptor.Chain): Response {
         val originalRequest = chain.request()
-        val noToken = originalRequest.header(HEADER_NO_SERVICE_TOKEN)
-
-        if (noToken == "true") {
-            return chain.proceed(removeNoTokenHeader(originalRequest))
-        }
 
         val token = runBlocking { tokenProvider.getAccessToken() }
-        if (token.isNullOrBlank()) {
-            return chain.proceed(originalRequest)
-        }
+        if (token.isNullOrBlank()) return chain.proceed(originalRequest)
 
         val newRequest = originalRequest.newBuilder()
             .addHeader(HEADER_AUTHORIZATION, "$TOKEN_PREFIX $token")
@@ -29,13 +21,7 @@ class AuthInterceptor(
         return chain.proceed(newRequest)
     }
 
-    private fun removeNoTokenHeader(request: Request): Request =
-        request.newBuilder()
-            .removeHeader(HEADER_NO_SERVICE_TOKEN)
-            .build()
-
     companion object {
-        private const val HEADER_NO_SERVICE_TOKEN = "No-Service-Token"
         private const val HEADER_AUTHORIZATION = "Authorization"
         private const val TOKEN_PREFIX = "Bearer"
     }

data/src/main/java/com/threegap/bitnagil/data/auth/datasourceimpl/AuthRemoteDataSourceImpl.kt

@@ -6,16 +6,18 @@ import com.threegap.bitnagil.data.auth.model.request.TermsAgreementRequest
 import com.threegap.bitnagil.data.auth.model.request.WithdrawalReasonRequest
 import com.threegap.bitnagil.data.auth.model.response.LoginResponse
 import com.threegap.bitnagil.data.auth.service.AuthService
+import com.threegap.bitnagil.data.auth.service.LoginService
 import com.threegap.bitnagil.data.common.safeApiCall
 import com.threegap.bitnagil.data.common.safeUnitApiCall
 import javax.inject.Inject
 
 class AuthRemoteDataSourceImpl @Inject constructor(
     private val authService: AuthService,
+    private val loginService: LoginService,
 ) : AuthRemoteDataSource {
     override suspend fun login(socialAccessToken: String, loginRequest: LoginRequest): Result<LoginResponse> =
         safeApiCall {
-            authService.postLogin(socialAccessToken, loginRequest)
+            loginService.postLogin(socialAccessToken, loginRequest)
         }
 
     override suspend fun submitAgreement(termsAgreementRequest: TermsAgreementRequest): Result<Unit> =
@@ -35,6 +37,6 @@ class AuthRemoteDataSourceImpl @Inject constructor(
 
     override suspend fun reissueToken(refreshToken: String): Result<LoginResponse> =
         safeApiCall {
-            authService.postReissueToken(refreshToken)
+            loginService.postReissueToken(refreshToken)
         }
 }

data/src/main/java/com/threegap/bitnagil/data/auth/service/AuthService.kt

@@ -1,23 +1,12 @@
 package com.threegap.bitnagil.data.auth.service
 
-import com.threegap.bitnagil.data.auth.model.request.LoginRequest
 import com.threegap.bitnagil.data.auth.model.request.TermsAgreementRequest
 import com.threegap.bitnagil.data.auth.model.request.WithdrawalReasonRequest
-import com.threegap.bitnagil.data.auth.model.response.LoginResponse
 import com.threegap.bitnagil.network.model.BaseResponse
 import retrofit2.http.Body
-import retrofit2.http.Header
-import retrofit2.http.Headers
 import retrofit2.http.POST
 
 interface AuthService {
-    @POST("/api/v1/auth/login")
-    @Headers("No-Service-Token: true")
-    suspend fun postLogin(
-        @Header("SocialAccessToken") socialAccessToken: String,
-        @Body loginRequest: LoginRequest,
-    ): BaseResponse<LoginResponse>
-
     @POST("/api/v1/auth/agreements")
     suspend fun submitAgreement(
         @Body termsAgreementRequest: TermsAgreementRequest,
@@ -28,10 +17,4 @@ 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<LoginResponse>
 }

data/src/main/java/com/threegap/bitnagil/data/auth/service/LoginService.kt

@@ -0,0 +1,23 @@
+package com.threegap.bitnagil.data.auth.service
+
+import com.threegap.bitnagil.data.auth.model.request.LoginRequest
+import com.threegap.bitnagil.data.auth.model.response.LoginResponse
+import com.threegap.bitnagil.network.model.BaseResponse
+import retrofit2.http.Body
+import retrofit2.http.Header
+import retrofit2.http.Headers
+import retrofit2.http.POST
+
+interface LoginService {
+    @POST("/api/v1/auth/login")
+    suspend fun postLogin(
+        @Header("SocialAccessToken") socialAccessToken: String,
+        @Body loginRequest: LoginRequest,
+    ): BaseResponse<LoginResponse>
+
+    @POST("/api/v1/auth/token/reissue")
+    @Headers("Auto-Login: true")
+    suspend fun postReissueToken(
+        @Header("Refresh-Token") refreshToken: String,
+    ): BaseResponse<LoginResponse>
+}