[Network] #155 - Login View 로직 수정, Refresh Token 로직추가 #156

shimseohyun · 2024-08-07T15:16:54Z

🔥 Pull requests

👷 작업한 내용

Login -> Onboarding -> Univ Select 으로 로직 변경했습니다. (나의 실수, 정말 미안)
HankkiList View Model에 Network Result 리펙토링이 안되어서 적용했습니다.
status code 204 대응
success 중 status code가 204로 오는 경우 response가 nil로 와서 decode 에러가 발생합니다. 해당 이슈 대응 로직을 작성했습니다. (후술)
401 에러(엑세스 토큰 만료)처리
401 에러 발생시 postReissue를 통해 엑세스 토큰을 재발급합니다.
이때 리프레시 토큰도 만료되었으면 application의 유저 정보를 지우고, 다시 로그인을 진행합니다.

🚨 참고 사항

status code 204 처리 방법

// NetworkResult.swift

extension NetworkResult {
    func handleNetworkResult(_ result: NetworkResult, onSuccess: ((T) -> Void)? = nil, onSuccessVoid: (() -> Void)? = nil) {
        switch result {
        case .success(let response):
            if let res = response {
                onSuccess?(res)
            } else if T.self == Void.self {
                onSuccessVoid?()
            } else {
                print("🚨 RESPONSE IS NIL 🚨")
            }
            
        case .unAuthorized:
            // 401 error
            // access token이 올바르지 않거나, 만료된 경우
            self.postReissue()
            
        default:
            // TODO: - 상세한 분기처리 필요 (기디 논의 필요)
            // 프로그램 로직 내부에 오류가 발생했을 경우, 모달창을 띄웁니다.
            if let windowScene = UIApplication.shared.connectedScenes.first as? UIWindowScene,
                let delegate = windowScene.delegate as? SceneDelegate,
                let rootViewController = delegate.window?.rootViewController {
                rootViewController.showAlert(titleText: "오류 발생",
                                             subText: result.stateDescription,
                                             primaryButtonText: "확인")
            }
        }
    }
}

204의 경우 서버에서 별도의 data를 보내지 않습니다. Void값이 오는데, emptyDTO를 쓰는 경우 타입이 같지 않아 디코딩 오류가 나게 됩니다.
NetworkResult로 선언 할 경우 response의 optional을 푸는 과정에서 예외처리 과정으로 빠지게 됩니다. 그래서, response 값이 nil이어도 호출할 수 있는 onSuccessVoid 클로저를 추가했습니다.

//MypageViewModel.swift
func deleteWithdraw(authorizationCode: String) {
        NetworkService.shared.authService.deleteWithdraw(authorizationCode: authorizationCode) { result in
            result.handleNetworkResult(result, onSuccessVoid: {
                print("🛠️ RESTART APPLICATION 🛠️ - WITHDRAW")
                UIApplication.resetApp()
            })
        }
    }

위와 같은 방식으로 결과가 204인경우 실행할 로직을 작성해주시면됩니다.

✅ Check List

Merge 대상 브랜치가 올바른가?
최종 코드가 에러 없이 잘 동작하는가?
전체 변경사항이 500줄을 넘지 않는가?

📟 관련 이슈

Resolved: [Network] Refresh Token 발급 로직 수정 #155

mcrkgus

훨씬 깔끔해졌구려 고생 많았소 최고입니다 서울우유 아기님

mcrkgus · 2024-08-08T14:54:02Z

Hankkijogbo/Hankkijogbo/Network/Base/NetworkResult.swift

            } else {
-                print(result)
+                print("🚨 RESPONSE IS NIL 🚨")


mcrkgus · 2024-08-08T14:57:07Z

Hankkijogbo/Hankkijogbo/Present/Login/ViewModel/LoginViewModel.swift

@@ -11,6 +11,27 @@ import UIKit
 final class LoginViewModel { }

 extension LoginViewModel {
+    func getUniversity() {


p3
깔끔하오 소녀 굳굳

mcrkgus · 2024-08-08T15:01:42Z

Hankkijogbo/Hankkijogbo/Present/Splash/SplashViewController.swift

    func getUniversity() {
-        // 사용자의 대학을 조회한다.
+        UserDefaults.standard.removeUniversity()


p3
온보딩에서 한 번 리셋 시키는건가유??

엇 이걸 논의 했어야했는데...!

access token 재발급 문제 때문에 splash 단계에서 api를 호출해서 access 토큰 상태를 한번 확인하는 로직이 있으면 좋겠다고 생각했습니다.
마침... university가 user defaults에 저정하는 것으로 변경되었기 때문에 + 혹시 오류로 꼬여서 user defaults에 대학 정보가 남아있을 수 있음으로 탈퇴, 로그아웃을 고려해 splash 단계에서 초기화 할 수 있다면 좋겠다고 생각했고...
-> 기존에 저장된 정보를 초기화하고 university api를 호출하며 그와 동시에 access token을 검사하는 것으로 로직을 짰습니다.

mcrkgus · 2024-08-08T15:02:15Z

Hankkijogbo/Hankkijogbo/Present/Splash/SplashViewController.swift

+                                                                  name: response.data.name,
+                                                                  longitude: response.data.longitude,
+                                                                  latitude: response.data.latitude)
+                UserDefaults.standard.saveUniversity(university)


p3
이 동무 애정하오

mcrkgus · 2024-08-08T15:02:53Z

Hankkijogbo/Hankkijogbo/Present/ZipList/View/ZipListViewController.swift

+        viewModel.postZipBatchDelete(requestBody: request) {
+            self.setIsEditMode()
+            self.dismiss(animated: false) {
+                self.viewModel.zipList = []
+                self.viewModel.getZipList()
+            }


p3
이슈 해결한 소녀 조아요

EunsuSeo01

고생하셨사옵나이다 아씨

EunsuSeo01 · 2024-08-08T16:40:15Z

Hankkijogbo/Hankkijogbo/Network/Base/NetworkResult.swift

                onSuccess?(res)
+            } else if T.self == Void.self {
+                onSuccessVoid?()
            } else {


Void.self라는 건 한번도 작성해본 적 없는데 이럴 때 쓰이는군요

EunsuSeo01 · 2024-08-08T17:06:27Z

Hankkijogbo/Hankkijogbo/Present/ZipList/View/ZipListViewController.swift

+            self.dismiss(animated: false) {
+                self.viewModel.zipList = []
+                self.viewModel.getZipList()
+            }


zipList를 비웠다가 다시 가져오는 로직인가요>?.?

네 맞습니다!
현재 플로우가 같은 페이지 내에서

수정 모드로 변경 + 삭제할 족보를 선택함

삭제 버튼을 누르면 팝업으로 뜸

팝업을 닫으면 삭제됨...

팝업을 닫음과 동시에 선택한 족보를 삭제된 리스트를 불러옴(zip list api를 한번 더 호출함)

인데, api 통신하는 동안 삭제 이전의 데이터가 떠서 그냥 아예 한번 배열을 초기화하고 다시 불러오는 방식으로 설정했습니다

shimseohyun added 6 commits August 7, 2024 10:39

[Network] #155 - Refresh Token 발급 기능 추가

45fa1ab

[Refactor] #155 - Splash 에서 Univ Select 까지 로직 수정

89807f5

[Fix] #155 - status code 204 예외 처리

ef39685

[Fix] #155 - post Zip Batch Delete 204 처리

c32342f

[Fix] #155 - delete Zip to Hankki 204 처리

446984f

[Refactor] #155 - Hankki List View Model 분기 처리

f352520

shimseohyun added ♻️ Refactor 전면 수정이 있을 때 사용 🤹🏻‍♀️ 서현 서현 공주 작업 🛜 Network API 연결 labels Aug 7, 2024

shimseohyun requested review from EunsuSeo01 and mcrkgus August 7, 2024 15:16

shimseohyun self-assigned this Aug 7, 2024

[Fix] #155 - presentUnivSelect 함수 변경

4d03c4e

mcrkgus approved these changes Aug 8, 2024

View reviewed changes

EunsuSeo01 approved these changes Aug 8, 2024

View reviewed changes

shimseohyun merged commit bb575ee into develop Aug 9, 2024

shimseohyun deleted the network/#155 branch August 9, 2024 00:44

shimseohyun mentioned this pull request Oct 10, 2024

[Feat] #269 - 둘러보기 로그인유저 식별 로직 추가 #270

Merged

3 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Network] #155 - Login View 로직 수정, Refresh Token 로직추가 #156

[Network] #155 - Login View 로직 수정, Refresh Token 로직추가 #156

shimseohyun commented Aug 7, 2024

mcrkgus left a comment

mcrkgus Aug 8, 2024

mcrkgus Aug 8, 2024

mcrkgus Aug 8, 2024

shimseohyun Aug 8, 2024

mcrkgus Aug 8, 2024

mcrkgus Aug 8, 2024

EunsuSeo01 left a comment

EunsuSeo01 Aug 8, 2024

EunsuSeo01 Aug 8, 2024

shimseohyun Aug 8, 2024 •

edited

Loading

[Network] #155 - Login View 로직 수정, Refresh Token 로직추가 #156

[Network] #155 - Login View 로직 수정, Refresh Token 로직추가 #156

Conversation

shimseohyun commented Aug 7, 2024

🔥 Pull requests

👷 작업한 내용

🚨 참고 사항

✅ Check List

📟 관련 이슈

mcrkgus 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

Choose a reason for hiding this comment

EunsuSeo01 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

shimseohyun Aug 8, 2024 • edited Loading

Choose a reason for hiding this comment

shimseohyun Aug 8, 2024 •

edited

Loading