Skip to content

[#464] Firebase 초기화보다 먼저 SDK 객체가 접근되는 순서를 개선한다#494

Merged
opficdev merged 1 commit into
developfrom
fix/#464-firebase-sdk
May 22, 2026
Merged

[#464] Firebase 초기화보다 먼저 SDK 객체가 접근되는 순서를 개선한다#494
opficdev merged 1 commit into
developfrom
fix/#464-firebase-sdk

Conversation

@opficdev
Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

Firebase 초기화 과정에서 configure() 이전에 Firebase 상태를 조회하며 경고가 발생하던 문제 방지

📝 작업 내용

📌 요약

  • FirebaseAppServiceImpl의 초기화 로직 단순화
  • FirebaseApp.app() 사전 조회 제거
  • 단일 실행 보장을 위한 내부 플래그 추가

🔍 상세

  • FirebaseAppServiceImpl에 정적 isConfigured 플래그 추가
  • configure() 호출 시 이미 초기화된 경우 조기 반환 처리
  • 기존 FirebaseApp.app() == nil 분기 제거
  • Firebase 상태 조회 없이 FirebaseApp.configure()만 단일 경로로 실행되도록 정리
  • configure() 이전 Firebase 접근으로 인한 startup 경고 방지

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this May 22, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

이 풀 리퀘스트는 Firebase가 중복으로 설정되는 것을 방지하기 위해 isConfigured 정적 변수를 추가하고 configure() 메서드 내에 가드 문을 도입했습니다. 리뷰를 통해 isConfigured 플래그의 스레드 안전성을 보장하고 Firebase SDK의 메인 스레드 호출 요구 사항을 충족하기 위해 configure() 메서드에 @MainActor를 추가하라는 피드백이 제공되었습니다.

@opficdev opficdev merged commit c57c3de into develop May 22, 2026
1 check passed
@opficdev opficdev deleted the fix/#464-firebase-sdk branch May 22, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Firebase 초기화보다 먼저 SDK 객체가 접근되는 순서를 개선한다

1 participant