[merge] be-develop 브랜치 내용 main 브랜치에 반영 #453
Merged
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
[chore] 이미지 URL과 함께 경로도 같이 반환하도록 수정
[docs] BE 기술 스택 내용 추가
[refactor] IP, 이메일 관련 기능 전부 주석 처리
[feat] 이미지 rollback 기능 구현
[fix] 타임라인 이미지 제거 시 예전 이미지 URL이 반환되는 문제 해결
[test] 깃허브 액션 파일 변경 및 생성
[fix] github action 스크립트 수정
추후 재사용 가능성 및 자료를 보존하기 위해 코드 완전 삭제 대신 주석 처리를 했다.
[chore] Papago API 제거
추후 애플 로그인 요청도 LoginRequestDto 인터페이스를 구현하도록 수정할 계획이라서 LoginRequestDto 인터페이스를 만들었다.
전략 패턴으로 작성하기 위해 소셜 로그인 interface와 구현체를 분리했다. 추후 애플 로그인도 전략 패턴으로 리팩토링할 예정이다.
기존 애플 로그인 관련 login과 refresh 메서드에는 (기존 메서드명 + Apple)을 추가했다.
LoginRequestDto 앞에 Social을 붙임에 따라 loginRequestDto 변수를 socialLoginRequestDto로 수정했다.
기존에 있던 CreateAuthRequestDto가 애플 로그인 요청 dto로 변경됐다.
기존 AuthService에 있던 decodedIdToken과 애플 로그인 로직을 AppleLoginStrategy 클래스로 옮겼다. 또한 컨트롤러에서 직접 헤더 값을 뽑아 매개변수로 전달하는 대신 모든 헤더를 맵에 담아 전달하는 방식으로 변경했다.
애플 계정 회원은 탈퇴할 때 idToken과 authorizationCode가 필요하기 때문에 Dto를 생성했다. 기존 카카오에는 요청 바디가 존재하지 않으므로, 단계별로 수정하기 위해 일단 withdraw와 withdraw2를 분리했다. 나중에 매개변수 등을 조정하여 withdraw2를 제거할 예정이다.
기존 카카오가 사용하던 withdraw 탈퇴 메서드에 매개변수에 SocialWithdrawRequestDto를 하였다.
dto의 경우, 별도의 인터페이스와 구현체가 필요하지 않다고 판단하여 기존 인터페이스를 클래스로 변경 및 파일명 변경을 진행했다. 이에 따라 기존 카카오/애플 구현체 dto를 제거하고, import 경로를 수정했다.
[chore] 워크플로우 삭제
[chore] workflow에서 actions/checkout 버전 변경
개발용 로그인을 먼저 등록함으로써 login/dev가 login/:social 라우터에 포함되지 않도록 함
[fix] 소셜 로그인과 개발용 로그인 라우터 순서 변경 & 경로 파라미터 관련 Swagger 추가
[docs] 아키텍처 이미지 변경 & WIKI 링크 추가
서버 내부 에러는 곧바로 500번대 에러로 재할당하여 정확한 에러 원인 파악이 어려웠다. 따라서 500번대 에러로 할당하기 전에 기존 에러 내용을 로그에 남기도록 수정했다.
[chore] 서버 내부 에러 로그 작성하도록 수정
0inn
approved these changes
Apr 16, 2024
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 요약
🌱 작업한 브랜치
📚 작업한 내용
백엔드 작업 내용을 main에 merge할게용~
📍 참고 사항
관련 이슈