[FEAT/#33] 소셜 로그인, 계정 변경하기 API의 oauth 인증 과정 변경

[sung-silver]

Related Issue 🚀

• closed [FEAT] 소셜 로그인, 계정 변경에 대해 회원가입과 동일한 로직을 사용하도록 변경 #33

Work Description ✏️

1. code와 token의 혼재

• 동규오빠가 바쁘신 것 같아서 그냥 token으로 모두 통일했습니다!

pr에도 code라고 적어주셨는데 프로젝트 코드에는 token으로 되어있더라고요 리뷰 과정에서 꼼꼼하게 확인하지 못해 죄송합니다

• 하나의 개념에 대해 여러개의 변수명이 사용되는 것은 좋지 못하다고 생각하여 이런 부분은 모두가 공유하면 좋을 것 같습니다!
• 슬랙에 먼저 공유한 후 명세서 소셜로그인/계정 변경하기 명세서에도 반영하겠습니다

2. SignUpService 내의 변수명 및 파라미터 순서 통일

• 이미 소셜 로그인, 계정 변경하기 서비스 로직에서 oauth 플랫폼으로부터 조회해온 식별자를 authPlatformId로 사용하고 있었기 때문에 identifier 변수명을 수정하여 통일하였습니다.
• 식별자라는 변수명도 좋지만 저희 서비스에서 authPlatformId라고 사용하기로 했기 때문에 이후 코드를 보았을 때 혼란을 야기할 수 있는 부분을 최소화하면 좋겠다는 생각이 들어 수정했습니다
• 또한 파라미터의 순서가 token, authPlatform 순서로 통일되면 좋겠다고 생각하여 관련된 메서드들의 파라미터 순서도 통일했습니다

methodA(String token, AuthPlatform authPlatform);
methodB(String authPlatformId, AuthPlatform authPlatform);

• 위와 같이 authPlatform이 뒤에 오는 방식으로 통일했습니다

3. OAuthService 인터페이스 재정의

• 제가 이전에 선언했던 메서드만 존재하고 getIdentifierByToken 메서드가 정의되어 있지 않아 이를 선언하고 사용되지 않는 메서드를 삭제했습니다

4. 사용하지 않는 클래스, 메서드 삭제

• Google/Apple Service/Client 클래스에서 사용되지 않는 로직을 삭제했습니다
• IdToken을 조회해오는 필요성이 없어짐에 따라 관련 dto 레코드도 삭제했습니다

5. GoogleAuthClient getPublicKeySet 예외처리 관심사 수정

• 예외가 apple로 되어 있어 google 예외를 추가하고 반영했습니다

PR Point 📸

• 아자아자 파이팅..~
• property 부분에서 저희가 id token을 조회해오지 않음에 따라 필요하지 않은 값들이 있는데요 .env 파일 수정도 필요하고 일단 배포가 우선인 것 같아서 이 부분은 PR을 따로 날리겠습니다! 현재 로직이 잘 되는지 테스트가 우선인 것 같아서요

[height]

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

[hyunw9]

팍 줄었네요 속썩이는 token.. 고생하셨습니다 ! 저는 딱히 변경할 점 없어보여요 !