Conversation
This file contains 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
…to refactor/498-FCM_서버_비동기처리_구현
Choi-JJunho
approved these changes
Jan 25, 2024
| private String keyScope; | ||
|
|
||
| private final RestTemplate restTemplate; | ||
| private static final String FCM_JSON_PATH = "config/pium-fcm.json"; |
There was a problem hiding this comment.
A
Value가 아니라 상수로 선언해서 약간 의아했는데 @PostConstruct 때문에 이렇게 한걸까요?
There was a problem hiding this comment.
ㅋㅋㅋ PR 올리고 할까 말까 고민중이었는데 제 마음을 잘 캐치해주셨네요
반영완류
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.
※ 해당 PR 및 커밋은 이건회(@rawfishthelgh )와 함께 몹 프로그래밍으로 작성했습니다
🔥 연관 이슈
🚀 작업 내용
FCM 서버에서 알림 처리를 하는 로직을 비동기로 처리할 수 있도록 변경했습니다.
기존에 사용하던
Restemplate을 제거하고FirebaseOptions객체를 이용해 서버 정보를 추가한 후FirebaseMessaging객체의sendAsync메서드를 통해 알림을 전송하게끔 구현했습니다.비동기 코드임에도 get을 호출해 response를 반환받은 이유는 비동기 처리 로직에서 future 타입의 결과를 받아 알림 요청의 성공 유무를 판단하기 위함입니다. 완전 비동기 호출보다 성능은 아~주 약간 떨어지지만 확실히 알림 전송 결과를 받아보는 것이 중요하기 때문에 위와 같이 결정했습니다.
곧 알림 실패 시 재전송하는 기능을 추가할 예정입니다 🥳
💬 리뷰 중점사항
이번에도 잘 부탁드립니다.