Feature/#90 token #93

ashwon12 · 2025-03-12T12:24:27Z

💡 Issue

Resolved: [Feature] 토큰 갱신 #90

🌱 Key changes

토큰 재발급 api 연동 ( Authenticator 생성 )
로그인 해제 & 유지
exception 추가

✅ To Reviewers

📸 스크린샷

x

Summary by CodeRabbit

새로운 기능
- 홈 화면에서 로그인 화면으로의 이동 경로가 추가되어, 필요 시 원활하게 로그인할 수 있습니다.
- 로그인 프로세스가 강화되어, 사용자의 토큰 상태를 확인한 후 자동으로 재로그인 처리가 이루어집니다.
UI 개선
- 공지사항 화면의 레이아웃을 최적화하여, 콘텐츠가 화면 전체 너비를 효과적으로 활용하도록 개선되었습니다.

coderabbitai · 2025-03-12T12:24:34Z

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

이 PR은 로그인, 인증, 네비게이션 및 UI 레이아웃 관련 여러 변경 사항을 포함합니다.

YappNavHost와 홈 네비게이션에 로그인 화면으로 이동할 수 있는 새로운 파라미터가 추가되었습니다.
UserRepository 인터페이스와 그 구현체에 사용자 액세스 토큰을 반환하는 메서드가 도입되었습니다.
토큰 재발급을 위한 AuthApi, TokenAuthenticator, ReissueTokenRequest 등의 기능이 추가되었습니다.
Home, Login, Notice 관련 화면에서 네비게이션 및 레이아웃 수정이 이뤄졌습니다.

Changes

파일 경로	변경 요약
`app/src/main/.../YappNavHost.kt`	`navigateLogin` 파라미터 추가 (로그인 네비게이션 지원)
`core/data-api/src/main/.../UserRepository.kt` `core/data/.../UserRepositoryImpl.kt`	사용자 액세스 토큰 반환을 위한 `getUserAccessToken()` 메서드 추가 및 구현
`core/data/.../AlarmRepositoryImpl.kt`	생성자에 `SecurityPreferences` 의존성 추가
`core/data/.../AuthApi.kt`	`reissueToken` 메서드 추가 (토큰 재발급 지원)
`core/data/.../NetworkModule.kt`	`TokenAuthenticator` 파라미터 추가 및 `.authenticator(tokenAuthenticator)` 호출 추가
`core/data/.../NoticesRequest.kt`	파일 삭제 (NoticeListRequest 클래스 제거)
`core/data/.../ReissueTokenRequest.kt`	`ReissueTokenRequest` 데이터 클래스 추가 (액세스 및 리프레시 토큰 포함)
`core/data/.../TokenAuthenticator.kt`	토큰 재발급 및 재인증을 위한 `TokenAuthenticator` 클래스 추가
`core/domain/.../CheckLoginStatusUseCase.kt`	로그인 상태 확인용 새로운 use case 추가 (`invoke` 메서드)
`core/model/.../YappExceptions.kt`	`TKN_0001`, `TKN_0002` 에러 코드 및 `InvalidTokenException` 클래스 추가
`core/ui/.../NoticeItem.kt`	`fillMaxWidth()` 수정자로 UI 컴포넌트 레이아웃 변경
`feature/home/.../HomeContract.kt` `feature/home/.../HomeScreen.kt` `feature/home/.../HomeViewModel.kt` `feature/home/.../HomeNavigation.kt`	홈 관련 네비게이션에 `NavigateToLogin` 사이드 이펙트 및 `navigateLogin` 파라미터 추가, 에러 발생 시 로그인 화면 이동 처리
`feature/login/.../LoginContract.kt` `feature/login/.../LoginScreen.kt` `feature/login/.../LoginViewModel.kt`	로그인 관련 인텐트 추가 (`EnterLoginScreen`), 로그인 화면 초기 진입 시 인텐트 전송 및 액세스 토큰 검증 로직 추가
`feature/notice/.../NoticeRoute.kt`	레이아웃 수정 (Box에 `Modifier.fillMaxSize()`, Column의 하단 패딩 제거)

Sequence Diagram(s)

Token 재발급 플로우

sequenceDiagram
    participant Client
    participant TokenAuthenticator
    participant SecurityPreferences
    participant AuthApi
    Client->>TokenAuthenticator: 401 응답 발생
    TokenAuthenticator->>SecurityPreferences: 현재 토큰 요청
    SecurityPreferences-->>TokenAuthenticator: accessToken, refreshToken 반환
    TokenAuthenticator->>AuthApi: ReissueTokenRequest 전송
    AuthApi-->>TokenAuthenticator: LoginResponse (새 토큰 제공)
    TokenAuthenticator->>SecurityPreferences: 새로운 토큰 저장
    TokenAuthenticator->>Client: 새로운 요청 생성 (업데이트된 토큰 포함)

로그인 네비게이션 플로우

sequenceDiagram
    participant HomeViewModel
    participant HomeScreen
    participant Navigator
    HomeViewModel->>HomeViewModel: loadHomeInfo() 실행
    HomeViewModel-->>HomeViewModel: InvalidTokenException catch
    HomeViewModel->>HomeScreen: postSideEffect(NavigateToLogin)
    HomeScreen->>Navigator: navigateToLogin 콜백 호출
    Navigator-->>HomeScreen: 로그인 화면으로 전환

Possibly related PRs

Feature/#78 login #83: YappNavHost의 네비게이션 업데이트와 직접 연결된 변경 사항입니다.
Feature/#79 profile #84: navigateLogin 파라미터 및 홈 네비게이션의 로그인 관련 업데이트와 관련된 PR입니다.
Feature/#69 homeview #77: homeNavGraph 내 navigateLogin 업데이트와 직접 관련된 코드 변경이 포함되어 있습니다.

Suggested labels

Feature, Design

Suggested reviewers

jinukeu

Poem

나는 작은 토끼, 코드 숲을 달리며
새로운 길을 찾았네, 로그인 길 열리니!
반짝반짝 토큰 재발급, 네비게이션 빛나고,
UI는 넓게 펼쳐져, 화면 가득 꿈을 담고,
코드 속 모험에, 우리 모두 한 걸음!
🐇✨

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

‼️ IMPORTANT
Auto-reply has been disabled for this repository in the CodeRabbit settings. The CodeRabbit bot will not respond to your replies unless it is explicitly tagged.

Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

jinukeu

요번 PR은 아키텍처 관점에서 위반된 사항이 좀 있어 코멘트 남겼습니다 ... !

토큰 갱신 작업 꼼꼼히 해주셔서 감사해요!

core/domain/src/main/java/com/yapp/domain/GetUserProfileUseCase.kt

feature/login/src/main/java/com/yapp/feature/login/LoginViewModel.kt

feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt

core/data/src/main/java/com/yapp/core/data/data/repository/UserRepositoryImpl.kt

ashwon12 · 2025-03-14T09:54:10Z

@coderabbitai codereview

coderabbitai · 2025-03-14T09:55:33Z

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get the list of files changed in the PR
git diff --name-status HEAD~1

# Get more detailed info about the PR
gh pr view 93 --json title,body,additions,deletions,changedFiles,files

Length of output: 2898

🏁 Script executed:

#!/bin/bash
# Get content of the new TokenAuthenticator which is a key part of the token reissue implementation
cat core/data/src/main/java/com/yapp/core/data/remote/retrofit/TokenAuthenticator.kt

Length of output: 1999

🏁 Script executed:

#!/bin/bash
# Get the ReissueTokenRequest model
cat core/data/src/main/java/com/yapp/core/data/remote/model/request/ReissueTokenRequest.kt

# Get the CheckLoginStatusUseCase
cat core/domain/src/main/java/com/yapp/domain/CheckLoginStatusUseCase.kt

# Get the AuthApi
cat core/data/src/main/java/com/yapp/core/data/remote/api/AuthApi.kt

# Get the login view model
cat feature/login/src/main/java/com/yapp/feature/login/LoginViewModel.kt

# Check the UserRepository and UserRepositoryImpl for token related methods
cat core/data-api/src/main/java/com/yapp/dataapi/UserRepository.kt
cat core/data/src/main/java/com/yapp/core/data/data/repository/UserRepositoryImpl.kt

Length of output: 7651

🏁 Script executed:

#!/bin/bash
# Get content of the NetworkModule to see how the TokenAuthenticator is integrated
cat core/data/src/main/java/com/yapp/core/data/remote/di/NetworkModule.kt

# Get SecurityPreferences implementation
find core -name "SecurityPreferences.kt" -exec cat {} \;

# Check the exceptions added in this PR
cat core/model/src/main/java/com/yapp/model/exceptions/YappExceptions.kt

# Examine HomeViewModel changes related to token handling
cat feature/home/src/main/java/com/yapp/feature/home/HomeViewModel.kt

Length of output: 9904

@ashwon12 PR #93에 대한 코드 리뷰를 진행하겠습니다.

코드 리뷰

1️⃣ TokenAuthenticator 구현

TokenAuthenticator 클래스를 통한 토큰 재발급 메커니즘이 잘 구현되어 있습니다.
OkHttp의 Authenticator 인터페이스를 활용하여 401 에러 발생 시 자동으로 토큰을 재발급받는 구조가 좋습니다.
다만, runBlocking을 사용하고 있는데, 이는 메인 스레드를 차단할 수 있습니다. 현재 컨텍스트에서는 필요한 선택일 수 있으나, 가능하면 비동기 대안을 고려해 보는 것이 좋을 것 같습니다.

2️⃣ 로그인 유지 기능

CheckLoginStatusUseCase를 통한 토큰 유효성 확인 로직이 깔끔합니다.
HomeViewModel과 LoginViewModel에서 토큰 상태에 따라 적절히 화면을 전환하는 로직이 잘 구현되어 있습니다.

3️⃣ 예외 처리

InvalidTokenException 추가로 토큰 관련 오류 처리가 명확해졌습니다.
예외 발생 시 로그인 화면으로 리디렉션하는 방식이 적절합니다.

4️⃣ 개선 제안

TokenAuthenticator에서 토큰 재발급 실패 시 에러 처리가 간단히 빈 문자열 반환으로만 되어 있습니다. 구체적인 에러 로깅이나 처리를 추가하는 것을 고려해보세요.
보안 측면에서 토큰 저장 방식(SecurityPreferences)은 적절해 보이나, 토큰 만료 시간 관리나 로그아웃 시 토큰 제거 등의 처리가 명확한지 확인해 보세요.

전체적으로 토큰 재발급과 로그인 유지 기능이 잘 구현되어 있습니다. 코드 구조가 깔끔하고 관심사 분리가 잘 되어 있습니다. 👍

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

jinukeu

고생하셨슴다!

jinukeu · 2025-03-14T09:56:28Z

core/data/src/main/java/com/yapp/core/data/data/repository/UserRepositoryImpl.kt

-    override suspend fun getUserAccessToken(): String {
-        return runBlocking { securityPreferences.flowAccessToken().firstOrNull() ?: "" }
+    override suspend fun getUserAccessToken(): Flow<String> {
+        return flow { emit(securityPreferences.flowAccessToken().firstOrNull()?: "") }


return securityPreferneces.flowAccessToken() 요렇게 안되던가용 ... !?

헉 됩니닷 .. 제가 타입을 안보고 그냥 다시 flow로 감쌀 생각만 했네요 ㅎ-ㅎ
수정할게요!

coderabbitai

Actionable comments posted: 5

♻️ Duplicate comments (1)

feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt (1)
103-107: 🧹 Nitpick (assertive)

네임드 아규먼트 사용을 권장합니다.

Preview 함수에서 파라미터를 전달할 때 네임드 아규먼트를 사용하면 코드의 가독성이 더 향상됩니다.
 HomeRoute(
-            navigateToLogin = {},
-            navigateToNotice = {},
-            navigateToSetting = {}
+            navigateToLogin = {},
+            navigateToNotice = {},
+            navigateToSetting = {}
 )

📜 Review details

Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8fce79d and b160844.

📒 Files selected for processing (21)

app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt (1 hunks)
core/data-api/src/main/java/com/yapp/dataapi/UserRepository.kt (1 hunks)
core/data/src/main/java/com/yapp/core/data/data/repository/AlarmRepositoryImpl.kt (2 hunks)
core/data/src/main/java/com/yapp/core/data/data/repository/UserRepositoryImpl.kt (1 hunks)
core/data/src/main/java/com/yapp/core/data/remote/api/AuthApi.kt (2 hunks)
core/data/src/main/java/com/yapp/core/data/remote/di/NetworkModule.kt (2 hunks)
core/data/src/main/java/com/yapp/core/data/remote/model/request/NoticesRequest.kt (0 hunks)
core/data/src/main/java/com/yapp/core/data/remote/model/request/ReissueTokenRequest.kt (1 hunks)
core/data/src/main/java/com/yapp/core/data/remote/retrofit/TokenAuthenticator.kt (1 hunks)
core/domain/src/main/java/com/yapp/domain/CheckLoginStatusUseCase.kt (1 hunks)
core/domain/src/main/java/com/yapp/domain/GetUserProfileUseCase.kt (0 hunks)
core/model/src/main/java/com/yapp/model/exceptions/YappExceptions.kt (2 hunks)
core/ui/src/main/java/com/yapp/core/ui/component/NoticeItem.kt (1 hunks)
feature/home/src/main/java/com/yapp/feature/home/HomeContract.kt (1 hunks)
feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt (3 hunks)
feature/home/src/main/java/com/yapp/feature/home/HomeViewModel.kt (2 hunks)
feature/home/src/main/java/com/yapp/feature/home/navigation/HomeNavigation.kt (1 hunks)
feature/login/src/main/java/com/yapp/feature/login/LoginContract.kt (1 hunks)
feature/login/src/main/java/com/yapp/feature/login/LoginScreen.kt (2 hunks)
feature/login/src/main/java/com/yapp/feature/login/LoginViewModel.kt (4 hunks)
feature/notice/src/main/java/com/yapp/feature/notice/notice/NoticeRoute.kt (1 hunks)

💤 Files with no reviewable changes (2)

core/domain/src/main/java/com/yapp/domain/GetUserProfileUseCase.kt
core/data/src/main/java/com/yapp/core/data/remote/model/request/NoticesRequest.kt

🧰 Additional context used

🪛 detekt (1.23.7)

core/data/src/main/java/com/yapp/core/data/remote/retrofit/TokenAuthenticator.kt

[warning] 35-35: The caught exception is too generic. Prefer catching specific exceptions to the case that is currently handled.

(detekt.exceptions.TooGenericExceptionCaught)

[warning] 35-35: The caught exception is swallowed. The original exception could be lost.

(detekt.exceptions.SwallowedException)

🔇 Additional comments (24)

feature/notice/src/main/java/com/yapp/feature/notice/notice/NoticeRoute.kt (2)

109-109: 레이아웃 구조 개선에 좋은 변경사항입니다.

Box 컴포넌트에 Modifier.fillMaxSize()를 추가함으로써 전체 화면을 차지하도록 하는 것은 비어있는 상태와 알림 목록 상태 모두에서 일관된 레이아웃을 제공합니다. 이렇게 하면 컨텐츠가 화면에 더 잘 맞게 표시됩니다.

113-113: 하단 패딩 제거가 레이아웃에 미치는 영향을 확인해주세요.

Column에서 padding(bottom = 52.dp)가 제거되었습니다. 이는 비어있는 상태의 UI가 화면을 채우는 방식을 변경합니다. 하지만 156-161줄에 있는 GradientBottom 컴포넌트가 하단에 그라데이션 효과를 제공하므로, 이 변경사항은 의도적인 디자인 결정으로 보입니다.

이 변경이 디자인팀의 요구사항과 일치하는지, 그리고 비어있는 상태의 UI가 모바일 기기에서 올바르게 표시되는지 확인해주세요.

core/ui/src/main/java/com/yapp/core/ui/component/NoticeItem.kt (1)

33-34: NoticeItem UI 향상: fillMaxWidth 적용

컴포넌트의 너비를 부모 컨테이너에 맞게 확장하여 UI 일관성을 개선했습니다. 이는 리스트 아이템에서 특히 중요한 속성으로, 화면에서 더 일관된 레이아웃을 제공합니다.

core/data-api/src/main/java/com/yapp/dataapi/UserRepository.kt (1)

7-7: 토큰 검증을 위한 적절한 메서드 추가

사용자 액세스 토큰을 Flow로 반환하는 메서드를 추가한 것은 토큰 재발급 기능 구현에 필수적입니다. Flow를 사용하면 토큰 상태 변화를 실시간으로 감지할 수 있어 적절한 선택입니다.

feature/home/src/main/java/com/yapp/feature/home/HomeContract.kt (1)

26-26: 로그인 화면 네비게이션 기능 추가

토큰 만료 또는 로그아웃 시 로그인 화면으로 이동할 수 있는 side effect를 추가한 것은 매우 적절합니다. 기존 네비게이션 패턴과 일관성을 유지하면서 사용자 인증 상태 관리를 강화했습니다.
core/data/src/main/java/com/yapp/core/data/remote/api/AuthApi.kt (1)

22-26:
❓ Verification inconclusive

토큰 재발급 API 엔드포인트 구현

토큰 재발급을 위한 API 엔드포인트를 추가한 것은 PR의 주요 목적인 인증 메커니즘 개선에 핵심적입니다. 엔드포인트 경로와 요청/응답 모델이 일관성 있게 설계되었습니다.

다만, TokenAuthenticator 구현 시 토큰 재발급 실패 케이스에 대한 에러 핸들링을 확실히 구현하셨는지 확인해 보세요.

🌐 Web query:
안드로이드 OkHttp Authenticator에서 토큰 재발급 실패 처리 모범 사례
💡 Result:

OkHttp의 Authenticator를 사용한 토큰 재발급 실패 처리의 모범 사례는 다음과 같이 정리할 수 있습니다:

1. 재시도 제한 메커니즘

최대 재시도 횟수 설정: 3회 이상 재시도 방지 [5][11]
if (responseCount(response) >= 3) return null
Exponential Backoff 적용: 2초 → 4초 → 8초 방식의 지연 재시도

2. 동시성 제어
synchronized(this) {
    // 토큰 갱신 로직
}
동일한 토큰 갱신 요청이 여러 스레드에서 발생할 경우 뮤텍스로 처리 [12]

3. 실패 시 사용자 처리 흐름

단계 처리 내용 구현 방법

1차 실패 토큰 갱신 시도 refreshToken API 호출 [10]

2차 실패 로컬 토큰 삭제 SharedPreferences/DataStore 초기화 [8]

최종 실패 로그인 화면 이동 startActivity(LoginActivity) [11]

4. 헤더 관리 주의사항
request.newBuilder()
    .removeHeader("Authorization")  // 기존 헤더 제거 필수
    .addHeader("Authorization", "Bearer $newToken")
중복 헤더 전송 방지를 위한 기존 인증 정보 삭제 [1][6]

5. 오류 유형 분기 처리
when (exception) {
    is RefreshTokenExpired -> navigateToLogin()
    is NetworkError -> showRetryDialog()
    is ServerError -> logErrorAndReport()
}
401 외에 403, 407 등의 프로토콜별 대응 추가 [9]

6. 의존성 주입 분리
// 토큰 갱신 전용 클라이언트
val authClient = OkHttpClient.Builder()
    .authenticator(null)  // 순환 의존성 방지
    .build()
API 호출용 클라이언트와 토큰 갱신용 클라이언트 분리 [8]

7. 비동기 처리 패턴
suspend fun refreshToken() = withContext(Dispatchers.IO) {
    // 코루틴을 이용한 비동기 처리
}
runBlocking 대신 코루틴 디스패처 사용 [10]

이 모범 사례들은 네트워크 요청 실패 시 무한 재시도 방지, 사용자 경험 개선, 시스템 안정성 향상을 위해 필수적인 요소들을 포함합니다. 실제 구현 시에는 앱의 인증 흐름과 서버 규격에 맞게 커스터마이징이 필요합니다[4][12].

Citations:

1: https://jinudmjournal.tistory.com/89

2: https://www.jaenung.net/tree/13293

3: https://velog.io/@onegold-11/Android-%ED%86%A0%ED%81%B0-%EC%B2%98%EB%A6%AC-%EC%A0%95%EB%A6%AC

4: https://modelmaker.tistory.com/entry/%EC%95%88%EB%93%9C%EB%A1%9C%EC%9D%B4%EB%93%9C-%EB%A7%8C%EB%A3%8C-%ED%86%A0%ED%81%B0-%EA%B0%B1%EC%8B%A0-%EC%9A%94%EC%B2%AD-api%EC%97%90-%ED%86%A0%ED%81%B0-%EC%82%BD%EC%9E%85-%EC%9E%90%EB%8F%99%ED%99%94-%EC%8B%9C%EC%8A%A4%ED%85%9C-%EA%B0%9C%EB%B0%9C%EA%B8%B0-3-Interceptor%EC%97%90%EC%84%9C-Token-%EC%82%BD%EC%9E%85%ED%95%98%EA%B8%B0-%ED%8E%B8-Android-Retrofit-Auto-Insert-Token-Auto-Refresh-Token

5: https://howtoprogram.xyz/2016/11/03/basic-authentication-okhttp-example/

6: https://uoohlz.tistory.com/entry/OkHttp-Interceptor%EB%A5%BC-%EC%9D%B4%EC%9A%A9%ED%95%B4-%ED%97%A4%EB%8D%94%EC%97%90-%ED%86%A0%ED%81%B0-%EB%B6%99%EC%9D%B4%EA%B8%B0Kotlin

7: https://sdusb.blogspot.com/2017/04/android-retrofit2-okhttpclient-method.html

8: https://jja2han.tistory.com/499

9: https://square.github.io/okhttp/3.x/okhttp/okhttp3/Authenticator.html

10: https://velog.io/@chuu1019/%EC%95%88%EB%93%9C%EB%A1%9C%EC%9D%B4%EB%93%9C%EC%97%90%EC%84%9C-Access-Token-%EB%A7%8C%EB%A3%8C-%EC%B2%98%EB%A6%AC%ED%95%98%EA%B8%B0-Feat.-dataStore

11: https://github.com/Commit451/FourOhOne

12: http://sangsoonam.github.io/2019/03/06/okhttp-how-to-refresh-access-token-efficiently.html

13: https://innodroid.com/blog/post/using-okhttp-authenticator

14: https://akaisun.tistory.com/73

15: https://vzio.tistory.com/206

16: https://velog.io/@trasalby/Retrofit2%EB%A5%BC-%EC%82%AC%EC%9A%A9%ED%95%B4-JWT-%ED%86%A0%ED%81%B0-%EC%9D%B8%EC%A6%9D%ED%95%98%EA%B8%B0-1
토큰 재발급 API 엔드포인트 구현 및 관련 오류 처리 검증 요청

토큰 재발급 API 인터페이스(core/data/src/main/java/com/yapp/core/data/remote/api/AuthApi.kt, 22-26)는 전반적으로 일관되게 설계되어 있습니다. 다만, TokenAuthenticator 구현 시, 토큰 갱신 실패 상황에 대비한 오류 핸들링이 모범 사례에 따라 아래 사항들(최대 재시도 횟수, Exponential Backoff, 동시성 제어, 실패 시 후속 처리)을 충분히 반영했는지 확인해 주시기 바랍니다.
core/data/src/main/java/com/yapp/core/data/remote/model/request/ReissueTokenRequest.kt (1)

5-9: 토큰 재발급 요청 모델 구현이 적절합니다

토큰 재발급 요청을 위한 데이터 클래스가 잘 구현되었습니다. accessToken과 refreshToken 두 필드를 포함하고 있으며, @serializable 애노테이션을 통해 직렬화 기능을 지원합니다. internal 가시성 수정자를 사용하여 모듈 내부로 접근을 제한한 것도 적절합니다.

feature/login/src/main/java/com/yapp/feature/login/LoginScreen.kt (1)

40-42: 로그인 화면 진입 시 토큰 상태 확인 로직이 적절하게 구현되었습니다

LaunchedEffect를 사용하여 로그인 화면이 처음 표시될 때 EnterLoginScreen 인텐트를 발행하는 방식이 깔끔합니다. 이를 통해 로그인 화면 진입 시 토큰 유효성을 확인하고 필요한 경우 세션 유지 처리가 가능합니다. Compose의 사이드 이펙트 처리 메커니즘을 적절히 활용한 좋은 구현입니다.

app/src/main/java/com/yapp/app/official/navigation/YappNavHost.kt (1)

42-45: 로그아웃 기능을 위한 네비게이션 경로가 적절하게 추가되었습니다

homeNavGraph에 navigateLogin 파라미터를 추가하여 홈 화면에서 로그인 화면으로 이동할 수 있는 기능이 구현되었습니다. clearBackStackNavOptions를 사용하여 백 스택을 정리함으로써 로그아웃 후에 인증이 필요한 화면으로 돌아갈 수 없도록 적절하게 처리되었습니다. 이는 로그아웃 기능 구현에 필수적인 요소입니다.

feature/login/src/main/java/com/yapp/feature/login/LoginContract.kt (1)

29-29: 로그인 화면 진입 인텐트가 적절하게 추가되었습니다

LoginIntent 인터페이스에 EnterLoginScreen 데이터 객체가 추가되어 로그인 화면 진입 시 토큰 상태를 확인하는 기능을 지원합니다. 이는 MVI 패턴에 따라 사용자 액션(화면 진입)을 나타내는 적절한 인텐트 구현입니다. LoginScreen에 추가된 LaunchedEffect와 함께 로그인 세션 유지 기능을 완성하는 중요한 요소입니다.

core/domain/src/main/java/com/yapp/domain/CheckLoginStatusUseCase.kt (1)

7-14: 잘 구현된 사용자 로그인 상태 검증 로직입니다!

CheckLoginStatusUseCase는 사용자의 로그인 상태를 확인하기 위한 명확한 책임을 가지고 있으며, 코드가 간결하고 이해하기 쉽습니다. 토큰의 유효성을 isNullOrBlank() 검사로 확인하는 방식이 적절합니다.

core/data/src/main/java/com/yapp/core/data/data/repository/AlarmRepositoryImpl.kt (1)

6-18: SecurityPreferences 의존성이 추가되었으나 사용되지 않습니다.

AlarmRepositoryImpl 클래스에 SecurityPreferences 의존성이 추가되었지만, 현재 코드에서는 이 의존성을 사용하지 않고 있습니다. 토큰 관련 기능을 위해 추가된 것으로 보이나, 미사용 의존성은 코드 가독성과 유지보수성을 저하시킬 수 있습니다.

현재 이 의존성이 필요한지 확인하고, 사용하지 않는다면 제거하거나 주석으로 향후 사용 계획을 명시하는 것이 좋습니다.

feature/home/src/main/java/com/yapp/feature/home/navigation/HomeNavigation.kt (1)

25-35: 로그인 화면 네비게이션 연결이 잘 추가되었습니다.

홈 네비게이션 그래프에 로그인 화면으로 이동하는 기능이 잘 추가되었습니다. 이는 토큰 관련 예외 처리 및 로그인 세션 유지 기능 구현에 필수적인 부분입니다.

feature/login/src/main/java/com/yapp/feature/login/LoginViewModel.kt (3)

7-7: CheckLoginStatusUseCase 의존성이 잘 추가되었습니다.

로그인 상태 확인을 위한 UseCase를 추가한 것이 좋은 접근 방식입니다. 이전 리뷰에서 언급된 UI 레이어와 데이터 레이어 간의 의존성 문제를 잘 해결했습니다.

80-80: EnterLoginScreen 인텐트 처리가 깔끔하게 구현되었습니다.

LoginIntent.EnterLoginScreen 인텐트를 추가하고 checkAccessToken 메서드로 연결한 구현이 명확합니다.

126-131: 토큰 체크 로직이 잘 구현되었습니다.

checkAccessToken 메서드가 간결하고 명확하게 구현되었습니다. viewModelScope를 사용하여 코루틴을 적절히 활용하고 있으며, 액세스 토큰 유효성에 따라 홈 화면으로 네비게이션하는 로직이 잘 작성되었습니다.

core/data/src/main/java/com/yapp/core/data/remote/di/NetworkModule.kt (2)

7-7: TokenAuthenticator import 및 관련 의존성이 잘 정리되었습니다.

TokenAuthenticator를 위한 import와 필요한 의존성들이 깔끔하게 정리되었습니다.

Also applies to: 13-14, 19-19

61-61: TokenAuthenticator가 OkHttpClient에 잘 통합되었습니다.

TokenAuthenticator를 OkHttpClient에 추가하여 토큰 재발급 메커니즘을 구현한 것이 좋습니다. 이제 401 오류 발생 시 자동으로 토큰을 재발급 받을 수 있습니다.

Also applies to: 66-66

core/model/src/main/java/com/yapp/model/exceptions/YappExceptions.kt (2)

19-21: 토큰 관련 에러 코드가 잘 추가되었습니다.

토큰 만료 및 유효성 검사를 위한 에러 코드(TKN_0001, TKN_0002)가 명확하게 추가되었습니다. 이를 통해 토큰 관련 예외 처리를 더 세분화할 수 있게 되었습니다.

34-34: InvalidTokenException 클래스가 잘 구현되었습니다.

비정상 토큰을 처리하기 위한 예외 클래스가 명확한 에러 메시지와 함께 구현되었습니다. 이 예외 클래스는 토큰 인증 실패 시 적절한 사용자 피드백을 제공하는 데 도움이 됩니다.

feature/home/src/main/java/com/yapp/feature/home/HomeScreen.kt (2)

32-32: navigateToLogin 파라미터가 잘 추가되었습니다.

HomeRoute 함수에 로그인 화면으로 이동하기 위한 콜백 함수가 적절히 추가되었습니다.

48-48: NavigateToLogin 사이드 이펙트 처리가 잘 구현되었습니다.

HomeSideEffect.NavigateToLogin 사이드 이펙트가 발생했을 때 navigateToLogin 콜백을 호출하는 로직이 잘 구현되었습니다. 이를 통해 토큰 관련 문제가 발생했을 때 사용자를 로그인 화면으로 적절히 리디렉션할 수 있습니다.

feature/home/src/main/java/com/yapp/feature/home/HomeViewModel.kt (2)

12-13: 명시적 예외 import 처리 좋습니다.

InvalidTokenException, UserNotFoundForEmailException 등 예외를 명시적으로 구분해 사용함으로써, 문제 발생 시 원인 파악이 훨씬 명확해졌습니다.

45-45: HomeIntent 처리 로직 확인했습니다.

EnterHomeScreen 인텐트 시 loadHomeInfo를 호출하여 필요한 정보를 로드하고, 후속 처리를 postSideEffect로 넘기는 구조는 유지보수성과 가시성을 높여줍니다.

core/data/src/main/java/com/yapp/core/data/data/repository/UserRepositoryImpl.kt

core/data/src/main/java/com/yapp/core/data/remote/retrofit/TokenAuthenticator.kt

coderabbitai · 2025-03-14T09:58:48Z

core/data/src/main/java/com/yapp/core/data/remote/retrofit/TokenAuthenticator.kt

+    private fun resetToken() {
+        runBlocking {
+            securityPreferences.setAccessToken("")
+            securityPreferences.setRefreshToken("")
+        }
+    }


🧹 Nitpick (assertive)

resetToken에서 runBlocking 사용 시점도 비동기 전환을 고려해 주세요.

토큰 리셋 시점에서 역시 runBlocking이 동작 스레드를 잠글 수 있으므로, 전체 구조가 허용한다면 비동기로 처리해 UI 응답성을 더욱 유지할 수 있도록 개선해 보세요.

A: resetToken 부분은 suspend function으로 잡으면 좋긴 하겠군요

feature/home/src/main/java/com/yapp/feature/home/HomeViewModel.kt

ashwon12 added 6 commits March 12, 2025 15:20

fix/#85 : 공지사항 아이템 디자인 수정

2f3ef03

feat/#90 : 토큰 재발급

99eb1e1

feat/#90 : 로그인 해제

3ed9967

feat/#90 : 로그인 유지

e1e79fd

refactor/#90 : 미사용 클래스 삭제

1302575

feat/#90 : 공통 에러 처리로 변경

ac0f701

ashwon12 requested a review from jinukeu March 12, 2025 12:24

ashwon12 self-assigned this Mar 12, 2025

jinukeu requested changes Mar 12, 2025

View reviewed changes

refactor/#90 : getAccessToken usecase 분리

b160844

ashwon12 requested a review from jinukeu March 14, 2025 09:53

ashwon12 added the CodeRabbit CodeRabbit 코드 리뷰 요청 label Mar 14, 2025

jinukeu approved these changes Mar 14, 2025

View reviewed changes

coderabbitai bot reviewed Mar 14, 2025

View reviewed changes

refactor/#90 : 코드 개선

1a01d63

ashwon12 merged commit e4f62b3 into develop Mar 14, 2025

ashwon12 deleted the feature/#90-token branch March 19, 2025 09:16

coderabbitai bot mentioned this pull request May 6, 2025

Feature/#166 공통 에러 처리, PullToRefresh Indicator 색상 변경 #169

Merged

단계	처리 내용	구현 방법
1차 실패	토큰 갱신 시도	`refreshToken` API 호출 [10]
2차 실패	로컬 토큰 삭제	`SharedPreferences`/`DataStore` 초기화 [8]
최종 실패	로그인 화면 이동	`startActivity(LoginActivity)` [11]

Feature/#90 token #93

Feature/#90 token #93

Uh oh!

Conversation

ashwon12 commented Mar 12, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

💡 Issue

🌱 Key changes

✅ To Reviewers

📸 스크린샷

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Mar 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Walkthrough

Changes

Sequence Diagram(s)

Token 재발급 플로우

로그인 네비게이션 플로우

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

jinukeu left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ashwon12 commented Mar 14, 2025

Uh oh!

coderabbitai bot commented Mar 14, 2025

코드 리뷰

1️⃣ TokenAuthenticator 구현

2️⃣ 로그인 유지 기능

3️⃣ 예외 처리

4️⃣ 개선 제안

Uh oh!

jinukeu left a comment

Choose a reason for hiding this comment

Uh oh!

jinukeu Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

ashwon12 Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

1. 재시도 제한 메커니즘

2. 동시성 제어

3. 실패 시 사용자 처리 흐름

4. 헤더 관리 주의사항

5. 오류 유형 분기 처리

6. 의존성 주입 분리

7. 비동기 처리 패턴

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

jinukeu Mar 14, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

ashwon12 commented Mar 12, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Mar 12, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)