[BE] feat: 유저 FCM 로직 변경(#471) #472

BGuga · 2023-09-27T04:33:36Z

📌 관련 이슈

closed: [BE] FCM 로직 변경 #471

✨ PR 세부 내용

로그인 시 중복 FCM 확인
회원가입 시 FCM 저장 ( 중복 확인 과정을 생략합니다 )
탈퇴 시 유저 FCM 삭제
모든 요청이 성공했을 시 예외를 발생시키지 않도록 변경

작업 완료했습니다.
MemberFCM table 에 (member_id, fcm_token) unique 제약 조건을 걸어주어

기존 쿼리

List<MemberFCM> memberFCM = memberFCMRepository.findByMemberId(memberId);

새로운 쿼리

Optional<MemberFCM> memberFCM = memberFCMRepository.findMemberFCMByMemberIdAndFcmToken(memberId, fcmToken);

가 인덱스 기반으로 쿼리를 날릴 수 있도록 변경하였고 unique 제약 조건으로 데이터 정합성도 맞췄습니다.

fcm 생성, 삭제에 대한 전체적인 흐름이
로그인 : fcm 존재 확인 후 저장 ✅
로그아웃: fcm ❌
회원탈퇴: fcm 삭제 ✅
회원가입: fcm 저장 ✅

github-actions · 2023-09-27T04:37:18Z

Unit Test Results

  75 files   75 suites 13s ⏱️
234 tests 234 ✔️ 0 💤 0 ❌
238 runs 238 ✔️ 0 💤 0 ❌

Results for commit 531f4a4.

♻️ This comment has been updated with latest results.

seokjin8678

리뷰 남겼습니다!

seokjin8678 · 2023-09-27T06:17:49Z

backend/src/main/java/com/festago/fcm/application/MemberFCMService.java

+        if (memberFCM.isPresent()) {
+            return;
+        }
+        memberFCMRepository.save(new MemberFCM(memberId, fcmToken));


Suggested change

if (memberFCM.isPresent()) {

return;

}

memberFCMRepository.save(new MemberFCM(memberId, fcmToken));

if (memberFCM.isEmpty()) {

memberFCMRepository.save(new MemberFCM(memberId, fcmToken));

}

다음과 같이 해도 좋을 것 같아요

seokjin8678 · 2023-09-27T06:22:56Z

backend/src/main/java/com/festago/fcm/application/FCMNotificationEventListener.java

+        if (failSend.isEmpty()) {
+            return;
+        }


Suggested change

if (failSend.isEmpty()) {

return;

}

if (batchResponse.getFailureCount() > 0) {

List<SendResponse> failSend = batchResponse.getResponses().stream()

.filter(sendResponse -> !sendResponse.isSuccessful())

.toList();

log.warn("member {} 에 대한 다음 요청들이 실패했습니다. {}", memberId, failSend);

}

다음과 같이 최적화 할 수 있을 것 같아요!

xxeol2

깔꼼하네요 👍

carsago · 2023-09-28T11:00:10Z

backend/src/main/java/com/festago/presentation/AuthController.java

+        String accessToken = response.accessToken();
+        String fcmToken = request.fcmToken();
+        if (response.isNew()) {
+            memberFCMService.saveNewMemberFCM(accessToken, fcmToken);
+            return;
+        }
+        memberFCMService.saveOriginMemberFCM(accessToken, fcmToken);


이 부분 isNew 여부 boolean으로 Service에 파라미터로 보내주고 Service에서 분기타는거 어떤가요??

seokjin8678

간단하게 리뷰 남겼습니다!
최신 dev 브랜치로 가져와서 롬복만 적용하면 될 것 같아요!
고생하셨습니다!

seokjin8678 · 2023-10-05T08:41:53Z

backend/src/main/java/com/festago/fcm/application/FCMNotificationEventListener.java

@@ -80,11 +77,11 @@ private AndroidNotification createAndroidNotification(String channelId) {
    }

    private void checkAllSuccess(BatchResponse batchResponse, Long memberId) {


이제 더 이상 check 하는 것이 아니니까 logFailResponses 정도로 해도 좋겠네요!

요 부분이 애쉬 PR에 반영돼어서 애쉬 PR을 머지한 후에 conflict 해결하면서 빠진 부분 채워 넣을게요!

public void sendMessages(List<Message> messages, FCMChannel channel) { try { BatchResponse response = firebaseMessaging.sendAll(messages); checkAllSuccess(response, channel); } catch (FirebaseMessagingException e) { log.warn("[FCM: {}] 전송 실패: {}", channel, e.getMessagingErrorCode()); } }

seokjin8678 · 2023-10-05T08:43:51Z

backend/src/main/java/com/festago/fcm/application/MemberFCMService.java

    @Transactional(readOnly = true)
    public MemberFCMsResponse findMemberFCM(Long memberId) {
        List<MemberFCM> memberFCM = memberFCMRepository.findByMemberId(memberId);
-        if (memberFCM.size() < LEAST_MEMBER_FCM) {
-            log.error("member {} 의 FCM 토큰이 발급되지 않았습니다.", memberId);
-            throw new InternalServerException(ErrorCode.FCM_NOT_FOUND);
+        if (memberFCM.isEmpty()) {
+            log.warn("member {} 의 FCM 토큰이 발급되지 않았습니다.", memberId);
        }
        return MemberFCMsResponse.from(memberFCM);
    }


추후 사용자의 FCM 등록 요청이 별도 API로 분리되게 된다면 해당 로그는 더 이상 찍을 필요가 없겠네요!

seokjin8678 · 2023-10-05T08:47:53Z

backend/src/main/java/com/festago/presentation/AuthController.java

    @DeleteMapping
    @SecurityRequirement(name = "bearerAuth")
    @Operation(description = "회원 탈퇴 요청을 보낸다.", summary = "유저 회원 탈퇴")
    public ResponseEntity<Void> deleteMember(@Member Long memberId) {
        authFacadeService.deleteMember(memberId);
+        memberFCMService.deleteMemberFCM(memberId);


FCM에 관한 의존성 분리를 하더라도 회원이 탈퇴하면 회원의 FCM을 삭제 해야하기 때문에 Auth 도매인이 FCM 도메인을 의존하게 되는 문제는 여전히 발생하겠네요.
이 부분은 DB의 Cascade로 풀어야 하나 고민이 드네요. (회원의 삭제가 Soft-delete라 전파가 될지도 모르겠네요. 😂)

로그인 과정에서 분리된 것이지 패키지 의존성으로 보면 그렇네요!!
저희가 eventListener를 사용하지 않았던 것이 결국 authFacadeService에 fcmToken 값이 들어갔기 때문인데 controller 에서 event 를 발행하면 authService에 fcmToken을 넘겨주지 않아도 되고 의존성도 분리되니 해당 방법은 어떠신가요?

carsago

굿

* fix: 유저의 fcm 요청이 모두 성공했을 때 예외를 발생시키지 않는다 * feat: 회원 탈퇴 시 memberFCM 삭제 및 로그인 시 memberFCM 저장 로직 구현 * chore: hard wrap 120 적용 * feat: 회원가입과 로그인 여부에 따라 FCM 로직을 분기한다 * feat: 회원가입 FCM 저장 로직 비동기 처리 * chore: 외래키 제약 조건 추가 분리 * feat: memberFCM 토큰이 없거나 메세징이 실패했을 경우에 log 만 작성하도록 변경 * refactor: 로직 개선 * test: 로직 변경에 따른 검증 내용 변경 * refactor: 신규 가입 여부에 따른 fcm 발행 관리를 Service 에게 위임 * chore: flyway 버전 변경

BGuga added 2 commits September 27, 2023 13:30

fix: 유저의 fcm 요청이 모두 성공했을 때 예외를 발생시키지 않는다

6a1c1f9

feat: 회원 탈퇴 시 memberFCM 삭제 및 로그인 시 memberFCM 저장 로직 구현

8ada1d6

BGuga linked an issue Sep 27, 2023 that may be closed by this pull request

[BE] FCM 로직 변경 #471

Closed

chore: hard wrap 120 적용

aea6950

feat: 회원가입과 로그인 여부에 따라 FCM 로직을 분기한다

1f6320f

seokjin8678 added the BE 백엔드에 관련된 작업 label Sep 27, 2023

BGuga self-assigned this Sep 27, 2023

github-actions bot requested review from carsago, seokjin8678 and xxeol2 September 27, 2023 05:50

feat: 회원가입 FCM 저장 로직 비동기 처리

37ee777

xxeol2 added the ☢️ DB 데이터베이스에 변경이 있는 작업 label Sep 27, 2023

BGuga added 2 commits September 27, 2023 14:57

chore: 외래키 제약 조건 추가 분리

2b998c0

feat: memberFCM 토큰이 없거나 메세징이 실패했을 경우에 log 만 작성하도록 변경

9be144f

seokjin8678 requested changes Sep 27, 2023

View reviewed changes

BGuga added 2 commits September 27, 2023 15:29

refactor: 로직 개선

84242d9

test: 로직 변경에 따른 검증 내용 변경

f1518b7

xxeol2 approved these changes Sep 27, 2023

View reviewed changes

carsago requested changes Sep 28, 2023

View reviewed changes

xxeol2 mentioned this pull request Oct 2, 2023

[BE] 입장 10분 전 푸시 알림 전송 기능 구현 (#483) #484

Closed

refactor: 신규 가입 여부에 따른 fcm 발행 관리를 Service 에게 위임

6c1db97

seokjin8678 requested changes Oct 5, 2023

View reviewed changes

seokjin8678 approved these changes Oct 5, 2023

View reviewed changes

carsago approved these changes Oct 6, 2023

View reviewed changes

chore: flyway 버전 변경

531f4a4

BGuga merged commit a5193e5 into dev Oct 11, 2023
3 checks passed

BGuga deleted the feat/#471 branch October 11, 2023 06:41

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] feat: 유저 FCM 로직 변경(#471) #472

[BE] feat: 유저 FCM 로직 변경(#471) #472

BGuga commented Sep 27, 2023 •

edited

Loading

github-actions bot commented Sep 27, 2023 •

edited

Loading

seokjin8678 left a comment

seokjin8678 Sep 27, 2023

seokjin8678 Sep 27, 2023

xxeol2 left a comment

carsago Sep 28, 2023

seokjin8678 left a comment

seokjin8678 Oct 5, 2023

BGuga Oct 5, 2023

seokjin8678 Oct 5, 2023

seokjin8678 Oct 5, 2023

BGuga Oct 5, 2023

carsago left a comment

-        if (failSend.isEmpty()) {
-            return;
-        }
+        if (batchResponse.getFailureCount() > 0) {
+            List<SendResponse> failSend = batchResponse.getResponses().stream()
+                .filter(sendResponse -> !sendResponse.isSuccessful())
+                .toList();
+            log.warn("member {} 에 대한 다음 요청들이 실패했습니다. {}", memberId, failSend);
+        }

		@@ -80,11 +77,11 @@ private AndroidNotification createAndroidNotification(String channelId) {
		}

		private void checkAllSuccess(BatchResponse batchResponse, Long memberId) {

[BE] feat: 유저 FCM 로직 변경(#471) #472

[BE] feat: 유저 FCM 로직 변경(#471) #472

Conversation

BGuga commented Sep 27, 2023 • edited Loading

📌 관련 이슈

✨ PR 세부 내용

github-actions bot commented Sep 27, 2023 • edited Loading

Unit Test Results

seokjin8678 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xxeol2 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

seokjin8678 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

carsago left a comment

Choose a reason for hiding this comment

BGuga commented Sep 27, 2023 •

edited

Loading

github-actions bot commented Sep 27, 2023 •

edited

Loading