회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #626

apptie · 2023-10-11T09:58:42Z

📄 작업 내용 요약

회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

flyway는 개발 서버에서 DB 트랜잭션 걸어놓고 테스트해서 동작함을 보장할 수 있습니다

📎 Issue 번호

closed 회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #625

kwonyj1022

수고하셨습니다.

kwonyj1022 · 2023-10-11T10:13:09Z

backend/ddang/src/main/java/com/ddang/ddang/user/domain/User.java

+    @Enumerated(EnumType.STRING)
+    private Oauth2Type oauth2Type;
+


질문

oauthId랑 OauthType을 embedded로 묶어서 관리하면 안되나요?

좋은 의견이네요 적용하겠습니다

kwonyj1022 · 2023-10-11T10:13:30Z

backend/ddang/src/main/resources/db/migration/V17__alter_user_oauth_type_tables.sql

+ALTER TABLE users ADD oauth2_type VARCHAR(10);
+UPDATE users SET oauth2_type = 'KAKAO' WHERE oauth2_type is null;
+ALTER TABLE users MODIFY oauth2_type VARCHAR(10) NOT NULL;


믿고 넘어갑니다

갑자기 불안해지네요

22 믿습니다 지토 🫡

저도 믿습니다 🙏

JJ503

대신 수정해 주셔서 감사합니다 지토 🙇‍♀️
고생 많으셨습니다!

JJ503 · 2023-10-12T05:49:34Z

...dang/src/main/java/com/ddang/ddang/authentication/presentation/AuthenticationController.java

-    @PostMapping("/withdrawal/{oauth2Type}")
+    @PostMapping("/withdrawal")


칭찬

대신 수정 진행해주셔서 감사합니다!
압도적 감사 🙇‍♀️

swonny

지토 고생 많으셨습니다!
질문 하나 남겼는데 이번 PR에서 중요하지 않은 부분이고, 주석은 제거해주시면 감사하겠습니다!

swonny · 2023-10-12T17:10:31Z

...dang/src/test/java/com/ddang/ddang/authentication/application/AuthenticationServiceTest.java

@@ -224,11 +224,12 @@ void setUp() {
    @Test
    void 가입한_회원이_탈퇴하는_경우_정상처리한다() throws InvalidWithdrawalException {
        // given
+        //given(tokenDecoder.decode(TokenType.ACCESS, anyString())).willReturn(Optional.of(사용자_id_클레임));


필수

주석 제거를 부탁드리겠습니다

삭제하려고 했었는데 까먹고 안했네요 수정하도록 하겠습니다

swonny · 2023-10-12T17:16:47Z

...d/ddang/src/main/java/com/ddang/ddang/user/infrastructure/persistence/JpaUserRepository.java

+        FROM User u
+        WHERE u.deleted = false AND u.oauthInformation.oauthId = :oauthId
+    """)
+    Optional<User> findByOauthId(final String oauthId);


질문

이건 제가 헷갈려서 드리는 질문입니다.
findPureUserByOauthId와 같이 변경해주지 않아도 괜찮은가요?
'경매 목록 조회 시 마감된 경매의 정렬 순서를 후순위로 변경' PR에서는 pureXX와 같이 변경해주셨길래 fetch join을 하는 항목이 없으면 pure를 무조건 붙이기로 했었는지 헷갈려서 질문 남깁니다!

일단 그 부분은 정확한 논의가 없던 것으로 기억합니다

근데 일단...pure기는 한데 fetch join하는 구간이 없어서 안붙이기는 했습니다...
fetch join을 하지 않는 레포지토리에서도 Pure로 명시를 해주는게 좋을까요?

* refactor: #605 경매 목록 조회 시 마감된 경매의 정렬 순서를 후순위로 변경 (#608) * refactor: 경매 목록 조회 시 마감된 경매의 정렬 순서를 후순위로 변경 * test: 정렬 조건 변경으로 인한 테스트 케이스 변경 * refactor: 복잡하지 않은 쿼리를 querydsl로 표현하지 않고 JPQL로 표현 * refactor: left outer join이 필요하지 않은 경우를 inner join으로 변경 * refactor: 모든 관계를 fetch join하는 Auction 조회 메서드 네이밍 변경 * feat: Auction만을 조회하는 메서드 추가 * refactor: Auction의 정보만이 필요한 비즈니스 로직에서 fetch join을 하지 않도록 변경 * chore: 배포 스크립트 의미 없는 break 키워드 삭제 * refactor: #613 경매 등록 시 경매 마감 시간 제한 변경 (#620) * refactor: 마감 시간을 미래만 적용할 수 있도록 제약조건 변경 * feat: 마감 시간 Custom Validator 추가 * refactor: 경매 등록 요청 시 마감 시간 검증을 Custom Validator으로 하도록 변경 * refactor: #609 소셜 로그인 시 최초 로그인(회원가입)한 회원인지 여부 반환 (#612) * refactor: 소셜 로그인 시 사용자가 최초 로그인을 하는지 여부를 의미하는 필드 추가 * docs: 문서 최신화 * refactor: 신규 회원가입을 했는지에 대한 네이밍을 명확하게 변경 * test: 실패하는 테스트 케이스 수정 * style: 개행 및 누락된 final 키워드 추가 * feat: #585 상호 평가 반영 스케줄링 (#602) * feat: 신뢰도 점수 타입을 Double -> double로 변경 * feat: 평가 저장 시 신뢰도를 갱신하지 않고 저장만 하도록 변경 * feat: 신뢰도 업데이트에 필요한 비즈니스 로직 추가 * feat: 신뢰도 업데이트 서비스 구현에 필요한 비즈니스 로직 보완 및 레포지토리 추가 * feat: 신뢰도 업데이트 서비스 추가 * feat: 스케줄링 설정 추가 * feat: flyway 스크립트 추가 * style: 개행 수정 * feat: 신뢰도 업데이트 이력을 관리하는 엔티티의 toString 대상 항목 추가 * refactor: 평가 관련 일급컬렉션에서 EqualsAndHashCode 재정의 제거 * test: 테스트코드에서 선언과 assertion 분리 * comment: 주석 제거 * refactor: #625 회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 (#626) * chore: flyway 버전 17 추가 * feat: User 도메인에 사용한 소셜 로그인 방식 필드 추가 * refactor: 회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 * refactor: Oauth2와 관련된 필드를 별도의 클래스로 분리 * chore: flyway 스크립트 버전 변경 * test: 의미 없는 테스트 mocking 삭제 * refactor: access token 재발급 시 refresh token도 재발급되도록 변경 * feat: #587 Q&A 알림 기능 구현 (#614) * feat: 답변 서비스에서 알림 이벤트 발행 * feat: 질문 서비스에서 알림 이벤트 발행 * feat: 질문, 답변 이벤트 리스너 추가 * chore: 수정 필요한 부분 주석 추가 * refactor: 이미지 절대 경로 컨트롤러에서 전송하도록 수정 * refactor: 이벤트 발행 시 별도의 DTO가 아닌 도메인 전달하도록 수정 * fix: 잘못 복제한 커밋 내역 복구 * refactor: 레포지토리 저장 엔티티를 이벤트 dto에 전달하도록 수정 * refactor: #649 Auction 관련 인프라 리펙토링 (#651) * refactor: Querydsl 기반의 Auction 인터페이스 제거 및 구현체에서 일반 클래스로 변환 * refactor: Querydsl 기반의 Auction 조회 인터페이스를 domain 패키지로 이동 * refactor: 경매 목록 조회 메서드 파라미터 순서 변경 * feat: 도메인 영역의 Repository 추가 * feat: 도메인 영역의 Repository 구현체 추가 * test: 누락된 테스트 케이스 추가 * feat: 경매 id가 존재하는지 여부를 반환하는 기능 추가 * refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경 * refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경 * rename: 경매 목록 조회 관련 테스트 패키지 분리 * remove: 사용하지 않는 경매 더미 데이터 초기화 로직 삭제 * test: fixture에서 구체적인 DB 구현 기술에 의존적이지 않도록 변경 * refactor: #648 Category 관련 인프라 리펙토링 (#650) * feat: 도메인 영역의 Repository 추가 * feat: 도메인 영역의 Repository 구현체 추가 * refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경 * refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경 * refactor: #645 Authentication 관련 인프라 리펙토링 (#647) * test: 누락된 테스트 케이스 추가 * feat: 도메인 영역의 Repository 추가 * feat: 도메인 영역의 Repository 구현체 추가 * refactor: Service에서 도메인 영역의 Repository를 사용하도록 변경 * test: when & then 주석 추가 * test: BlackListToken의 도메인 인터페이스 Fixture에서 구체적인 인프라 구현 방식을 알지 못하도록 변경 * test: fixture 메서드 네이밍 변경 * refactor: #655 User 관련 인프라 리팩토링 (#661) * feat: 도메인 영역의 사용자 Repository 추가 * feat: 도메인 영역의 사용자 Repository 구현체 추가 * test: 누락된 테스트 케이스 추가 * refactor: Service에서 도메인 영역의 사용자 Repository를 사용하도록 변경 * refactor: id로 사용자 조회 시 삭제된 값은 조회하지 않도록 수정 * feat: 도메인 영역의 사용자 신뢰도 Repository 추가 * feat: 도메인 영역의 사용자 신뢰도 Repository 구현체 추가 * test: 누락된 테스트 케이스 추가 * refactor: Service에서 도메인 영역의 사용자 신뢰도 Repository를 사용하도록 변경 * refactor: 사용자 신뢰도 조회 시 사용자도 함께 조회되도록 수정 * fix: 사용자 레포지토리 구현체에서 잘못된 메서드를 호출해서 테스트를 실패하는 현상 해결 * test: 테스트 실패 해결 * refactor: 사용자 신뢰도 엔티티에서 cascade 옵션 수정 * feat: 도메인 영역의 신뢰도 업데이트 기록 Repository 추가 * feat: 도메인 영역의 신뢰도 업데이트 기록 Repository 구현체 추가 * test: 누락된 테스트 케이스 추가 * refactor: Service에서 도메인 영역의 신뢰도 업데이트 기록 Repository를 사용하도록 변경 * refactor: 픽스처에서 Jpa 사용자 Repository 의존 제거 * refactor: 픽스처에서 Jpa 사용자 신뢰도 Repository 의존 제거 * refactor: 픽스처에서 Jpa 신뢰도 업데이트 기록 Repository 의존 제거 * style: 개행 추가 * refactor: 누락된 final 추가 * test: jpa 저장 테스트 시 불필요한 엔티티 매니저 호출 제거 * refactor: 누락된 final 추가 * refactor: RepositoryImpl 테스트에서 Jpa 의존 제거 * refactor: #656 ChatRoom 관련 인프라 리팩토링 (#663) * feat: 도메인 영역의 채팅방 Repository 추가 * refactor: 불필요하게 Querydsl을 사용한 코드를 JPQL을 사용하도록 변경 * feat: 도메인 영역의 채팅방 Repository 구현체 추가 * refactor: Service에서 도메인 영역의 채팅방 Repository를 사용하도록 변경 * refactor: 픽스처에서 Jpa 채팅방 Repository 의존 제거 * feat: 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository 추가 * feat: 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository 구현체 추가 * refactor: Service에서 도메인 영역의 채팅방 정보(채팅방, 이미지) dto를 반환하는 Repository를 사용하도록 변경 * feat: 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository 추가 * feat: 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository 구현체 추가 * feat: Service에서 도메인 영역의 채팅방 정보(채팅방, 메시지, 이미지) dto를 반환하는 Repository를 사용하도록 변경 * style: 개행 추가 * refactor: 누락된 final 추가 * ci: 브랜치 최신화 과정에서 누락된 사항 추가 및 충돌 해결 * refactor: #658 Region 관련 인프라 리팩토링 (#665) * feat: 도메인 영역의 지역 Repository 추가 * feat: 도메인 영역의 지역 Repository 구현체 추가 * test: 누락된 테스트 케이스 추가 * refactor: Service에서 도메인 영역의 지역 Repository를 사용하도록 변경 * test: 테스트에서 Jpa 의존 제거 * test: 테스트에서 Jpa 의존 제거 * style: 공백 추가 * refactor: 누락된 final 추가 * refactor: #657 Review 관련 인프라 리팩토링 (#664) * feat: 도메인 영역의 평가 Repository 추가 * feat: 도메인 영역의 평가 Repository 구현체 추가 * refactor: Service에서 도메인 영역의 평가 Repository를 사용하도록 변경 * test: 픽스처에서 Jpa 의존 제거 * test: 테스트 실패 케이스 해결 * refactor: 누락된 final 추가 * ci: 브랜치 최신화 과정에서 누락된 사항 추가 및 충돌 해결 * refactor: #659 입찰 목록 조회 API 명세 변경에 따른 수정 (#667) * refactor: 입찰 목록 조회 시 배열이 그대로 반환되도록 수정 * docs: 문서화 최신화 * refactor: 체이닝 개행 수정 * refactor: #660 Q&A 목록 조회 API 명세 변경에 따른 수정 (#669) * refactor: 컨벤션에 따른 개행 수정 * feat: 질문과 답변 조회 시 질문 작성자인지 판단하는 값 추가 * refactor: 컨벤션에 따른 개행 수정 * refactor: 메서드 순서 변경 * fix: 충돌로 발생한 문제 해결 * refactor: import 와일드카드 제거 * fix: #680 신규 회원가입이 안되는 버그 수정 (#681) * fix: 신규 회원 가입 시 null이 될 수 없는 필드를 초기화하지 않아 레코드 저장에 실패하던 문제 수정 * test: 테스트 케이스 검증 부분 추가 및 잘못된 테스트 케이스 수정 * test: Fixture 네이밍과 테스트 메서드 네이밍을 명확하게 변경 * refactor: 초기 신뢰도를 상수로 초기화 * refactor: 초기 신뢰도 상수 변경 * refactor: #670 Image 관련 인프라 리팩토링 (#678) * refactor: 변수명을 명확하게 수정 * feat: 도메인 영역에 경매 이미지 repository 추가 * feat: 도메인 영역에 경매 이미지 repository의 구현체 추가 * refactor: service에서 도메인 영역의 repository를 사용하도록 변경 * feat: 도메인 영역에 프로필 이미지 repository 추가 * feat: 도메인 영역에 프로필 이미지 repository의 구현체 추가 * refactor: service에서 도메인 영역의 프로필 이미지 repository를 사용하도록 변경 * refactor: 변수명을 의도에 맞게 수정 * refactor: 레파지토리 어노테이션 누락 추가 * test: final 추가 * test: final 추가 * refactor: import 와일드카드 제거 * refactor: 쿼리문 명령어를 대문자로 수정 * refactor: 세팅 메서드명 수정 * refactor: 잘못된 클래스명 수정 * refactor: #671 Bid 관련 인프라 리팩토링 (#684) * feat: 도메인 영역에 입찰 repository 추가 * refactor: 메서드 네이밍과 파라미터 변수명을 명확하게 수정 * feat: 도메인 영역에 입찰 repository의 구현체 추가 * refactor: service에서 도메인 영역의 입찰 repository를 사용하도록 변경 * refactor: 레파지토리 어노테이션 누락 추가 * test: 테스트 영역에서 도메인 영역의 입찰 repository를 사용하도록 변경 * refactor: 쿼리문 명령어를 대문자로 수정 * refactor: 레파지토리에서 null이 아닌 Optional을 통해 반환되도록 수정 * test: 도메인 영역에 입찰 repository의 구현체 테스트 추가 * test: 테스트 영역에서 도메인 영역의 repository를 사용하도록 변경 * refactor: #672 Report 관련 인프라 리팩토링 (#685) * feat: 도메인 영역에 경매 신고 repository 추가 * refactor: EntityGraph를 jpql로 변경 * refactor: EntityGraph를 jpql로 변경 * feat: 도메인 영역에 경매 신고 repository 구현체 추가 * refactor: service에서 도메인 영역의 경매 신고 repository를 사용하도록 변경 * test: 테스트에서 도메인 영역의 경매 신고 repository를 사용하도록 변경 * feat: 도메인 영역에 채팅방 신고 repository 추가 * refactor: EntityGraph를 jpql로 변경 * feat: 도메인 영역에 쪽지방 신고 repository 구현체 추가 * test: 테스트 실패 해결 * test: 픽스처 클래스 네이밍 수정 * feat: 도메인 영역에 질문 신고 repository 추가 * refactor: EntityGraph를 jpql로 변경 * test: 테스트에서 도메인 영역의 질문 신고 repository를 사용하도록 변경 * refactor: service에서 도메인 영역의 쪽지방 신고 repository를 사용하도록 변경 * refactor: service에서 도메인 영역의 질문 신고 repository를 사용하도록 변경 * refactor: import 와일드카드 제거 * refactor: EntityGraph를 jpql로 변경 * feat: 도메인 영역에 답변 신고 repository 추가 * feat: 도메인 영역에 답변 신고 repository 구현체 추가 * refactor: service에서 도메인 영역의 답변 신고 repository를 사용하도록 변경 * test: 테스트에서 도메인 영역의 신고관련 repository를 사용하도록 변경 * refactor: import 와일드카드 제거 * test: 테스트 실패 문제 해결 * test: final 추가 * refactor: 쿼리문 명령어를 대문자로 수정 * refactor: 클래스명 수정 * refactor: 필드 순서 변경 및 불필효한 필드 제거 * test: 테스트 영역에서 도메인 영역의 repository를 사용하도록 변경 * refactor: #673 Q&A 관련 인프라 리팩토링 (#686) * feat: 도메인 영역에 질문 repository 추가 * refactor: EntityGraph를 jpql로 변경 * refactor: 사용하지 않는 클래스 제거 * refactor: 잘못된 쿼리문 수정 * feat: 도메인 영역에 질문 repository 구현체 추가 * refactor: service에서 도메인 영역의 질문 repository를 사용하도록 변경 * test: 테스트에서 도메인 영역의 질문 repository를 사용하도록 변경 * refactor: 사용하지 않는 메서드 제거 * refactor: 쿼리문 명령는 대문자로 수정 * feat: 도메인 영역에 답변 repository 추가 * feat: 도메인 영역에 답변 repository 구현체 추가 * test: 테스트에서 도메인 영역의 답변 repository를 사용하도록 변경 * test: 테스트 실패 문제 해결 * fix: 잘못된 쿼리 조건 수정 * test: 불필요한 로직 제거 * refactor: final 누락 추가 * feat: #687 닉네임 수정 시 중복 예외 처리 추가 (#688) * feat: 이름 수정시 이미 존재하는 이름에 대한 예외처리 추가 * feat: 예외 처리 추가 * refactor: #689 Message 인프라 리팩토링 (#694) * feat: 도메인 영역에 메시지 레포지토리 추가 * refactor: 메서드명에서 레포지토리명과 중복되는 부분 제거 * fix: 불필요한 메서드 제거 * refactor: 도메인 메시지 레포지토의 구현체 추가 * refactor: 서비스에서 도메인 레포지토리 사용하도록 변경 * refactor: 클래스명 변경에 따른 테스트 네이밍 변경 * test: 도메인 메시지 레포지토리 구현체 테스트 추가 * refactor: 변수명 명확하게 변경 * refactor: 누락된 then절 추가 * test: 테스트에서 도메인 영역의 메시지 레포지토리 사용하도록 수정 * refactor: 테스트 픽스처에서 도메인 영역의 메시지 레포지토리를 사용하도록 변경 * refactor: 불필요한 주석 삭제 * refactor: 누락된 SuppressWarnings 어노테이션 추가 * refactor: 불필요한 import문 삭제 및 사용되지 않는 필드 지역변수로 변경 * style: 불필요한 개행 제거 * refactor: #692 경매 상세 조회 시 로그인한 사용자가 해당 경매의 최고 입찰자인지 여부를 반환하도록 응답 형식 변경 (#695) * refactor: 경매 상세 조회 api 요청 시 로그인한 사용자가 해당 경매의 최종 입찰자인지 여부를 반환하도록 변경 * docs: 문서 최신화 * style: 개행 변경 * refactor: #690 Device 인프라 리팩토링 (#696) * feat: 도메인 영역 기기토큰 레포지토리 추가 * feat: 도메인 기기토큰 레포지토리의 구현체 추가 * feat: 기기토큰 레포지토리 메서드 중 누락된 메서드 추가 * refactor: 서비스 레이어에서 도메인 영역 기기토큰 레포지토리 사용하도록 변경 * refactor: 누락된 final 추가 * test: 테스트에서 도메인 영역의 기기토큰 레포지토리 사용하도록 변경 * refactor: 레포지토리명에 맞춰 변수명 변경 * refactor: 테스트 픽스처에서 도메인 영역의 사용자 레포지토리 사용하도록 변경 * test: 기기토큰 레포지토리 구현체 테스트 추가 * chore: 질문에 대한 todo 주석 추가 * style: 개행 추가 * refactor: jpaQueryFactory 파라미터명 통일 * refactor: 필드 순서 변경 * refactor: given절 추가 * refactor: jpa 레포지토리 저장 테스트 메서드에서 엔티티 매니저 사용하는 코드 삭제 * refactor: 불필요한 필드 삭제 * refactor: 테스트에서 도메인 레포지토리 사용하도록 변경 * refactor: 불필요한 todo 삭제 * refactor: #697 질문과 답변 조회 시 삭제된 답변은 보이지 않도록 수정 (#698) * test: 질문이 삭제된 경우 조회되지 않음을 나타내는 테스트 추가 * refactor: 답변이 삭제된 경우 null이 반환되도록 수정 * fix: 답변이 null인 경우에 대한 처리 추가 (#700) * feat: #573 탈퇴 시 사용자 닉네임 랜덤 값으로 수정 (#606) * feat: 탈퇴 시 닉네임을 랜덤 UUID로 변경하는 로직 추가 * feat: 탈퇴 시 프로필 이미지 삭제 기능 추가 * refactor: 가입 시 이미지를 기본 이미지가 아닌 null로 전달 * feat: 프로필 이미지 아이디가 null이라면 기본 이미지 url을 반환 기능 추가 * refactor: import 와일드카드 제거 * feat: 이미지가 null인 경우 기본 이미지를 반환하는 기능 추가 * test: 픽스처 네이밍을 명확하게 수정 * fix: 충돌 문제 해결 * fix: #701 답변 삭제 시 다시 답변을 달지 못하는 문제 해결 (#704) * feat: 답변에 작성자 필드 추가 * feat: 필드 추가에 대한 flyway 스크립트 추가 * feat: 답변 삭제 시 질문과의 연관관계 끊기 기능 추가 * refactor: 문자열 상수 처리 * test: import 와일드카드 제거 * fix: 문제가 발생할 flyway 스크립트 수정 * fix: flyway 스크립트 문제 해결 (#706) * feat: #705 경매 관련 탈퇴 로직 추가 (#710) * refactor: 경매 상세 조회 시 탈퇴한 사용자라면 알 수 없음으로 출력되도록 수정 * refactor: ToString과 EqualsAndHashCode 설정 수정 * feat: 사용자가 등록한 경매, 마지막인 경매 중 현재 진행 중인 경매가 있는지 확인하는 기능 레파지토리에 추가 * feat: 사용자가 등록한 경매, 마지막인 경매 중 현재 진행 중인 경매가 있는지 확인하는 기능 서비스에 추가 * refactor: jpql에서 querydsl로 변경 * test: 컨트롤러 테스트 추가 * test: import 와일드카드 제거 * refactor: querydsl을 jpa 레파지토리로 이동 --------- Co-authored-by: apptie <[email protected]> Co-authored-by: 권예진 <[email protected]> Co-authored-by: 최승원 Merry <[email protected]>

apptie added 3 commits October 11, 2023 18:54

chore: flyway 버전 17 추가

f370cdc

feat: User 도메인에 사용한 소셜 로그인 방식 필드 추가

664d5b0

refactor: 회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경

f352cc1

apptie added refactor 기존 기능에 변경이 없는 구현 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Oct 11, 2023

apptie added this to the 최종 데모데이 milestone Oct 11, 2023

apptie requested review from JJ503, swonny and kwonyj1022 October 11, 2023 09:58

apptie self-assigned this Oct 11, 2023

kwonyj1022 approved these changes Oct 11, 2023

View reviewed changes

refactor: Oauth2와 관련된 필드를 별도의 클래스로 분리

394869b

JJ503 approved these changes Oct 12, 2023

View reviewed changes

chore: flyway 스크립트 버전 변경

f8b754f

swonny approved these changes Oct 12, 2023

View reviewed changes

apptie added 3 commits October 13, 2023 09:44

test: 의미 없는 테스트 mocking 삭제

c1ef727

ci: 브랜치 최신화

7d3cad2

refactor: access token 재발급 시 refresh token도 재발급되도록 변경

7011751

apptie merged commit 7143025 into develop-be Oct 13, 2023
1 check passed

apptie deleted the refactor/625 branch October 13, 2023 08:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #626

회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #626

apptie commented Oct 11, 2023

kwonyj1022 left a comment

kwonyj1022 Oct 11, 2023

apptie Oct 11, 2023

kwonyj1022 Oct 11, 2023

apptie Oct 11, 2023

JJ503 Oct 12, 2023

swonny Oct 12, 2023

JJ503 left a comment

JJ503 Oct 12, 2023

apptie Oct 12, 2023

swonny left a comment

swonny Oct 12, 2023

apptie Oct 13, 2023

swonny Oct 12, 2023

apptie Oct 13, 2023

		@PostMapping("/withdrawal/{oauth2Type}")
		@PostMapping("/withdrawal")

회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #626

회원 탈퇴 시 소셜 로그인 방식을 DB에서 조회하도록 변경 #626

Conversation

apptie commented Oct 11, 2023

📄 작업 내용 요약

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

📎 Issue 번호

kwonyj1022 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

JJ503 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

칭찬

Choose a reason for hiding this comment

swonny 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