Skip to content

[#619] FCM Token을 해제하지 못할 시 로그아웃에 실패하는 현상을 해결한다#622

Merged
opficdev merged 4 commits into
developfrom
fix/#619-logout
Jun 16, 2026
Merged

[#619] FCM Token을 해제하지 못할 시 로그아웃에 실패하는 현상을 해결한다#622
opficdev merged 4 commits into
developfrom
fix/#619-logout

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • 로그인 취소와 로그아웃 중 FCM 토큰 해제 실패가 실제 장애가 아닌데도 오류로 전파되거나 Crashlytics에 기록되는 문제를 방지

📝 작업 내용

📌 요약

  • 소셜 로그인 취소를 Infra 레이어에서 비오류 흐름으로 처리
  • FCM 토큰이 없는 상태의 로그아웃에서 deleteToken() 호출 생략
  • FCM 토큰 해제 실패가 로그아웃 실패나 Crashlytics 기록으로 이어지지 않도록 방어

🔍 상세

  • AuthenticationService.signIn()link()가 취소 흐름을 표현할 수 있도록 결과 기반으로 조정
  • Google, GitHub, Apple 로그인 취소 시 Infra 구현체에서 에러를 던지지 않고 흐름 종료
  • 로그아웃 시 messaging.fcmToken이 있을 때만 FCM 토큰 삭제 수행
  • FCM 토큰 삭제 실패는 로컬 로그만 남기고 Auth.auth().signOut()은 계속 진행
  • 실제 FirebaseAuth 로그아웃 실패는 기존처럼 Crashlytics 기록 유지
  • 관련 Data, Domain, Presentation 테스트를 변경된 취소 흐름에 맞게 갱신

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Jun 16, 2026
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

@opficdev

Copy link
Copy Markdown
Owner Author

/gemini review

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

이번 PR은 소셜 로그인 및 계정 연동 시 취소 처리를 에러 던지기 대신 Bool 또는 옵셔널 반환값으로 처리하도록 리팩토링하고, FCM 토큰 삭제 시의 에러 처리를 개선하였습니다. 리뷰어는 FCM 토큰 삭제 실패는 안전하게 처리되었으나, Firestore에서 FCM 토큰을 삭제하는 과정(infoRef.updateData)에서 에러가 발생할 경우 여전히 로그아웃 전체가 실패할 수 있으므로 이에 대한 추가적인 예외 처리를 권장하고 있습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@opficdev opficdev added the qa TestFlight에 배포 label Jun 16, 2026
@opficdev opficdev merged commit 94a5dc0 into develop Jun 16, 2026
5 checks passed
@opficdev opficdev deleted the fix/#619-logout branch June 16, 2026 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

qa TestFlight에 배포

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FCM Token을 해제하지 못할 시 로그아웃에 실패하는 현상을 해결한다

1 participant