[#600] 맥에서 소셜 로그인이 안되는 현상을 해결한다#603
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 PR에서는 FCM 토큰 동기화 로직을 개선하여 앱 활성화 시점에 토큰을 동기화하고, NotificationCenter를 통해 AppDelegate와 PushMessagingService 간의 결합도를 낮췄습니다. 이에 대한 피드백으로, 백그라운드 스레드에서 메인 스레드 API가 호출되어 발생할 수 있는 스레드 안전성 문제를 방지하기 위해 DispatchQueue.main.async를 사용할 것과, 불필요한 중첩 Task 생성을 피하기 위해 메서드를 async로 리팩토링할 것을 제안했습니다. 또한, Void? 타입을 일반 Void로 단순화하여 코드 가독성을 높이도록 권장했습니다.
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 연관된 이슈
🎯 의도
fcmToken을 backfill하지 못하는 문제 해결 목적📝 작업 내용
📌 요약
fcmToken이 없어도 인증 성공 처리하도록 수정AppDelegate에서 FCM 동기화 트리거를 이벤트 발행 중심으로 분리FCMTokenSyncHandler에서 APNs 수신 이후 현재 FCM token backfill 흐름 처리FCMTokenSyncHandler동작 테스트 추가1.2반영🔍 상세
messaging.token()강제 호출 제거AuthDataResponse.fcmToken을 optional로 변경fcmToken이 없으면 tokens 문서에 값을 쓰지 않도록 처리UIScene.didActivateNotification시점에 FCM 동기화 요청 이벤트 발행didReceiveAPNSToken이벤트 발행FCMTokenSyncHandler가흐름 담당
PushMessagingService는 권한 확인, APNs token 연결, FCM token fetch 책임만 유지FCMTokenSyncHandlerTests로 current token sync, nil no-op, refreshed token sync, APNs token backfill 케이스 검증 추가📸 영상 / 이미지 (Optional)