Skip to content

πŸ”€ :: (#120) - μ„Έμ…˜ 만료 μ‹œ 둜그인 ν™”λ©΄μœΌλ‘œ 정상 μ „ν™˜λ˜μ§€ μ•ŠλŠ” 문제 μˆ˜μ • - #121

Merged
ryusuye0n merged 2 commits into
developfrom
fix/#120-auth-session-expiry-handling
Sep 10, 2026
Merged

πŸ”€ :: (#120) - μ„Έμ…˜ 만료 μ‹œ 둜그인 ν™”λ©΄μœΌλ‘œ 정상 μ „ν™˜λ˜μ§€ μ•ŠλŠ” 문제 μˆ˜μ •#121
ryusuye0n merged 2 commits into
developfrom
fix/#120-auth-session-expiry-handling

Conversation

@ryusuye0n

@ryusuye0n ryusuye0n commented Sep 4, 2026

Copy link
Copy Markdown
Member

πŸ’‘ κ°œμš”

둜그인 ν›„ λͺ‡ μ‹œκ°„ λ’€ 앱을 λ‹€μ‹œ 켜면 "읡λͺ…λ‹˜"으둜 ν‘œμ‹œλ˜κ³  둜그인 ν™”λ©΄μœΌλ‘œλ„ μ•ˆ λ„˜μ–΄κ°€λ©° 먹톡이 λ˜λŠ” 문제λ₯Ό μˆ˜μ •ν–ˆμŠ΅λ‹ˆλ‹€.

πŸ”— κ΄€λ ¨ 이슈

Closes #120

πŸ“ƒ μž‘μ—…λ‚΄μš©

  • μ„Έμ…˜ μ’…λ£Œ 처리 λˆ„λ½: AuthInterceptor.onErrorμ—μ„œ 401 응닡에 refresh token이 없을 λ•Œ onSessionExpiredλ₯Ό ν˜ΈμΆœν•˜μ§€ μ•Šκ³  κ·Έλƒ₯ λ„˜μ–΄κ°€λ˜ 버그. 이제 μ¦‰μ‹œ μ„Έμ…˜μ„ μ’…λ£Œν•΄ 둜그인 ν™”λ©΄μœΌλ‘œ 정상 μ „ν™˜λ©λ‹ˆλ‹€.
  • κ°±μ‹  레이슀 μ»¨λ””μ…˜: FloodingAuthedClientκ°€ API ν˜ΈμΆœλΆ€λ§ˆλ‹€(8κ³³) λ³„λ„μ˜ Dio/AuthInterceptor μΈμŠ€ν„΄μŠ€λ₯Ό 생성해, 각자 λ…λ¦½λœ single-flight κ°±μ‹  락을 κ°–κ³  μžˆμ—ˆμŠ΅λ‹ˆλ‹€. λ™μ‹œμ— μ—¬λŸ¬ μš”μ²­μ΄ 401을 λ°›μœΌλ©΄ μ„œλ‘œ λ‹€λ₯Έ(이미 νšŒμ „λœ) refresh token으둜 쀑볡 갱신을 μ‹œλ„ν•΄ μ‹€νŒ¨ν•  수 μžˆμ—ˆμŠ΅λ‹ˆλ‹€. μ•± μ „μ—­μ—μ„œ κ³΅μœ ν•˜λŠ” SharedTokenRefresherλ₯Ό λ„μž…ν•΄ 갱신을 1회둜 μ‘°μœ¨ν•˜λ„λ‘ μˆ˜μ •ν–ˆμŠ΅λ‹ˆλ‹€.

πŸ” ν…ŒμŠ€νŠΈ 방법

  • test/core/network/auth_interceptor_test.dart μ‹ κ·œ μΆ”κ°€ β€” 정상 κ°±μ‹ , refresh token μ—†μŒ, κ°±μ‹  μ‹€νŒ¨, λ™μ‹œ μš”μ²­ single-flight μΌ€μ΄μŠ€ 컀버
  • flutter analyze, flutter test 전체(91개) 톡과 확인

πŸ–ΌοΈ μŠ€ν¬λ¦°μƒ·

N/A (λ„€νŠΈμ›Œν¬ 계측 둜직 μˆ˜μ •)

πŸ™‹β€β™‚οΈ μ§ˆλ¬Έμ‚¬ν•­

βœ… 체크리슀트

  • μ½”λ“œκ°€ μ •μƒμ μœΌλ‘œ 컴파일되고 μ‹€ν–‰λ˜λŠ”μ§€ ν™•μΈν–ˆμŠ΅λ‹ˆλ‹€.
  • λΆˆν•„μš”ν•œ μ½”λ“œκ°€ μ—†λŠ”μ§€ ν™•μΈν–ˆμŠ΅λ‹ˆλ‹€.
  • μ½”λ“œ μŠ€νƒ€μΌ κ°€μ΄λ“œλ₯Ό μ€€μˆ˜ν–ˆμŠ΅λ‹ˆλ‹€.
  • κΈ°μ‘΄ κΈ°λŠ₯이 μ •μƒμ μœΌλ‘œ μž‘λ™ν•˜λŠ”μ§€ ν™•μΈν–ˆμŠ΅λ‹ˆλ‹€.

@ryusuye0n ryusuye0n self-assigned this Sep 4, 2026
@ryusuye0n ryusuye0n added the ⚠️ Priority: High μš°μ„ μˆœμœ„ 상 label Sep 4, 2026

@eum018 eum018 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μˆ˜κ³ ν•˜μ…¨μŠ΅λ‹ˆλ‹€

@aiden30015 aiden30015 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

μˆ˜κ³ ν•˜μ…¨μŠ΅λ‹ˆλ‹€

@ryusuye0n
ryusuye0n merged commit a6cfe56 into develop Sep 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚠️ Priority: High μš°μ„ μˆœμœ„ 상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

μ„Έμ…˜ 만료 μ‹œ 둜그인 ν™”λ©΄μœΌλ‘œ λͺ» λŒμ•„κ°€κ³  λ©ˆμΆ°μžˆλŠ” 문제

3 participants