Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX/#204] firebase / storage 업로드 #205

Merged
merged 3 commits into from
Dec 4, 2024
Merged

Conversation

TaewoongR
Copy link
Collaborator

📍 Work Description

  • firebase storage에 요청하는 에피소드 업데이트 함수 수정 (withTimeout 제거)
  • firebase app check 추가

📸 Screenshot



📢 To Reviewers

  • 특정 기기에서 이미지가 업로드가 안되어 무한 대기에 걸리는 현상이 있었습니다.
  • kotlinx에서 제공하는 withTimeout를 사용하여 5초 이내에 실행되지 않으면 예외를 띄우는 코루틴을 firebase-storage 업로드 요청 로직에 감싸서 사용하니 api35 의 기기에서는 무한대기가 발생했습니다. 이를 제거하여 실행 가능하게 했습니다.
  • play store 배포시 인증된 유저만 firebase에 접근가능할 수 있도록 app check 를 설정했습니다.
  • app check 는 디버그를 지원하지 않지만 unenforced 상태로 만들어 배포 전에도 접근 가능하게 할 수 있습니다.

⏲️Time

- 3시간

- 특정 api(35) 기기에서 업데이트 불가 현상 수정
- withTimeout 제거
- play_store 앱 배포시 검증된 유저만 firestore, storage에 접근 가능하게 설정
@TaewoongR TaewoongR added the 🛠️ fix 버그 및 오류 수정 label Dec 3, 2024
@TaewoongR TaewoongR self-assigned this Dec 3, 2024
@TaewoongR TaewoongR requested a review from a team as a code owner December 3, 2024 16:38
@TaewoongR TaewoongR requested review from Ameri-Kano, haeti-dev and shinythinking and removed request for a team December 3, 2024 16:38
Copy link
Collaborator

@shinythinking shinythinking left a comment

Choose a reason for hiding this comment

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

감사합니다!
꽤 심각한 오류가 있었군요!
새로 알게된 개념이었을텐데 대단하십니다!

Copy link
Collaborator

@Ameri-Kano Ameri-Kano left a comment

Choose a reason for hiding this comment

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

일부 API에서만 문제가 발생하는 건 신기하네요...

Comment on lines 187 to 189
val createdBy = database.collection("user").document(episodeModel.createdBy)
val group = database.collection("group").document(episodeModel.group)
database.document("episode/${episodeModel.id}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

정의된 상수 사용해주셔도 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

적용해볼게요!

@TaewoongR TaewoongR merged commit df4257e into develop Dec 4, 2024
2 checks passed
@TaewoongR TaewoongR deleted the fix/firebase-storage branch December 4, 2024 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ fix 버그 및 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] Firebase / storage 업로드 문제 수정
4 participants