[🔀 merge] [4차] 코드 리펙토링 사항 메인 서버에 적용 #242

junggyo1020 · 2025-05-11T06:09:24Z

📄 Work Description

[🔀 merge] [4차] 코드 리펙토링 사항을 메인 서버에 적용합니다.

[🔥 !hotfix] [3차] 스테이징 서버 마이그레이션 진행

- `org.terning.terningserver.jwt` → `org.terning.terningserver.auth.jwt`로 변경 - 패키지 구조 정리로 가독성과 유지보수성 향상

- `AppleServiceImpl`을 `AppleAuthTokenValidator`로 변경하여 책임 분리 - `@Service` 대신 `@Component`를 사용하여 역할 명확화 - 패키지 경로를 `org.terning.terningserver.auth.application.social.apple`로 이동 - Apple 공개 키 조회 및 검증 로직을 `AppleAuthTokenValidator`에서 처리하도록 수정 - `ValueConfig` 활용하여 설정값을 일관되게 적용

- `org.terning.terningserver.controller.AuthController` → `org.terning.terningserver.auth.api.AuthController`로 이동 - `AuthService` 패키지 경로 변경에 따라 import 수정 - `signIn` 메서드에서 토큰 유효성 검사 로직 추가 - `sign-up/filter` 요청 처리 방식 변경 (`registerUserFilter`로 메서드명 변경 및 내부 로직 수정) - 패키지 구조 정리로 가독성과 유지보수성 향상

- `org.terning.terningserver.service.AuthService`를 `org.terning.terningserver.auth.application.AuthService`로 이동 - 기존 `AuthService` 인터페이스 제거 및 구현체로 통합 - `AuthSignInService`, `AuthSignUpService`, `AuthSignOutService`, `AuthWithdrawService`, `AuthReissueService`를 주입하여 역할 분리 - `registerFilterWithUser` 메서드 추가 (`createAndSaveFilter` 및 `connectFilterToUser` 통합) - `@Transactional(readOnly = true)` 적용하여 읽기 작업 최적화 - 패키지 구조 정리로 유지보수성 및 코드 가독성 향상

- `KakaoServiceImpl` → `KakaoAuthTokenValidator`로 클래스명 변경 - 패키지를 `org.terning.terningserver.auth.application.social.kakao`로 이동 - `@Service` 대신 `@Component`를 사용하여 역할 명확화 - `getKakaoData` → `extractKakaoId`로 메서드명 변경하여 의미 명확화 - `Authorization` 헤더에 `Bearer ` 접두어 추가 - 패키지 구조 정리 및 역할 분리를 통해 유지보수성 향상

- `AppleService` 및 `KakaoService` 인터페이스 제거 - `SocialAuthServiceManager` 클래스 추가하여 소셜 인증 서비스 관리 역할 수행 - `SocialAuthProvider` 기반으로 `AuthType`에 따른 적절한 인증 서비스 제공 - `create` 정적 메서드를 통해 `SocialAuthProvider` 리스트를 매핑하여 초기화 - 지원되지 않는 소셜 로그인 타입 처리 로직 추가 - 인증 서비스 구조 정리를 통해 확장성과 유지보수성 향상

- `AuthServiceImpl` 클래스 제거 및 역할별 서비스로 분리 - `AppleService`, `KakaoService` 제거 및 `SocialAuthServiceManager`로 통합 - `signIn`, `signUp`, `signOut`, `withdraw`, `reissueToken` 메서드를 개별 서비스로 위임 - `AuthSignInService`, `AuthSignUpService`, `AuthSignOutService`, `AuthWithdrawService`, `AuthReissueService` 추가 - `getAuthId` 메서드를 `SocialAuthServiceManager`에서 관리하도록 변경 - `WebhookService`를 회원가입 시 디스코드 알림 전송하는 역할로 유지 - 코드 구조 개선으로 유지보수성 및 확장성 향상

- 사용되지 않는 `SignUpFilterResponseDto` 제거 - 코드 정리로 불필요한 클래스 제거하여 유지보수성 향상

- `@EventListener`를 추가하여 `UserSignedUpEvent` 발생 시 자동으로 웹훅 알림 전송 - `sendDiscordNotification` 메서드를 `private`으로 변경하여 캡슐화 강화 - 불필요한 개행 제거 및 코드 정리로 가독성 향상

- `UserSignedUpEvent` 클래스 추가하여 회원가입 이벤트 관리 - `private` 생성자 및 정적 팩토리 메서드(`of`)를 사용하여 객체 생성 제어 - 회원가입 후 관련 서비스에서 이벤트를 활용할 수 있도록 설계

- `AuthReissueService` 인터페이스 추가하여 토큰 재발급 로직 추상화 - `reissueToken` 메서드를 정의하여 액세스 토큰 재발급 기능 제공 - 인증 서비스 구조 개선을 위한 인터페이스 분리

- `AuthReissueService` 인터페이스 구현체 `AuthReissueServiceImpl` 추가 - `reissueToken` 메서드를 통해 리프레시 토큰을 기반으로 액세스 토큰 재발급 - `JwtTokenProvider`를 활용하여 새로운 액세스 토큰 생성 - 리프레시 토큰 검증 후 사용자 조회 및 예외 처리 (`FAILED_TOKEN_REISSUE`) - 인증 서비스 구조 개선을 위해 책임 분리

- `AuthSignInService` 인터페이스 구현하여 로그인 로직 추상화 - `signIn` 메서드를 정의하여 사용자 로그인 기능 제공 - 인증 서비스 구조 개선을 위해 역할 분리

- `AuthSignInService` 인터페이스 구현 - `SocialAuthServiceManager`를 활용하여 소셜 로그인 제공자를 통한 `authId` 추출 - 사용자 정보를 조회하여 로그인 처리, 존재하지 않을 경우 신규 사용자 응답 반환 - `JwtTokenProvider`를 이용해 액세스 토큰 및 리프레시 토큰 생성 - 인증 서비스 역할을 분리하여 코드의 확장성과 유지보수성 향상

- `AuthSignOutService` 인터페이스 구현하여 로그아웃 로직 추상화 - `signOut` 메서드를 정의하여 사용자 로그아웃 기능 제공 - 인증 서비스 구조 개선을 위해 역할 분리

- `AuthSignOutService` 인터페이스 구현체 `AuthSignOutServiceImpl` 구현 - `signOut` 메서드를 통해 사용자의 리프레시 토큰 초기화 - 사용자 ID로 `UserRepository`에서 조회, 존재하지 않을 경우 예외 발생 (`INVALID_USER`) - 인증 서비스 역할을 분리하여 코드의 유지보수성과 확장성 향상

- `AuthSignUpService` 인터페이스 구현하여 회원가입 로직 추상화 - `signUp` 메서드를 정의하여 사용자의 회원가입 기능 제공 - `registerFilterWithUser` 메서드를 추가하여 사용자 필터 등록 기능 구현 - 인증 서비스 구조 개선을 위해 역할 분리

- `AuthSignUpService` 인터페이스 구현체 `AuthSignUpServiceImpl` 구현 - `signUp` 메서드를 통해 회원가입 처리 및 토큰 발급 - `registerFilterWithUser` 메서드를 통해 사용자 필터 등록 기능 구현 - `ApplicationEventPublisher`를 활용하여 `UserSignedUpEvent` 이벤트 발행 - JWT 토큰 생성 및 사용자 정보 저장 로직 분리하여 유지보수성 향상

- `SocialAuthProvider` 인터페이스 구현하여 애플 로그인 처리 - `AppleAuthTokenValidator`를 사용하여 `authAccessToken`에서 `authId` 추출 - `supports` 메서드를 통해 `AuthType.APPLE` 지원 명시 - 소셜 로그인 서비스 구조 분리로 유지보수성 및 확장성 향상

- `SocialAuthProvider` 인터페이스 구현하여 카카오 로그인 처리 - `KakaoAuthTokenValidator`를 사용하여 `authAccessToken`에서 `authId` 추출 - `supports` 메서드를 통해 `AuthType.KAKAO` 지원 명시 - 소셜 로그인 서비스 구조 분리로 유지보수성 및 확장성 향상

- `SocialAuthProvider` 인터페이스 구현하여 소셜 로그인 로직 추상화 - `getAuthId` 메서드를 통해 인증 토큰에서 사용자 ID 추출 기능 정의 - `supports` 메서드를 통해 지원하는 `AuthType` 반환 - 소셜 로그인 서비스의 확장성과 유지보수성을 고려한 구조 설계

`AuthWithdrawService` 인터페이스 구현하여 회원 탈퇴 로직 추상화 - `withdraw` 메서드를 정의하여 사용자 계정 삭제 기능 제공 - 인증 서비스 구조 개선을 위해 역할 분리

- `AuthWithdrawService` 인터페이스 구현체 `AuthWithdrawServiceImpl` 구현 - `withdraw` 메서드를 통해 사용자 계정 삭제 처리 - `UserRepository`와 `UserService`를 활용하여 사용자 삭제 로직 처리 - 사용자 존재하지 않을 경우 예외 발생 (`INVALID_USER`) - 인증 서비스 역할을 분리하여 코드의 유지보수성과 확장성 향상

- `SocialAuthServiceManager`를 생성하는 `socialAuthServiceFactory` 빈 정의 - `List<SocialAuthProvider>`를 활용하여 다양한 소셜 인증 프로바이더 설정 - 인증 서비스의 확장성과 유연성 향상을 위한 설정 추가

[♻️ refactor/#196] Auth application 계층 구조 개선

JwtTokenProvider 클래스의 역할을 분리하여 JwtTokenIssuer를 도입하고, JWT 서명 및 클레임 생성 책임을 별도의 클래스로 이동시킴. 이로 인해 JWT 관련 로직이 더 명확하게 분리되고, 유지보수성이 향상됨.

기존 response 설정 방식 대신 예외를 던지는 방식으로 변경하여 예외 처리를 일관되게 유지. JwtException을 활용해 JWT 관련 예외를 명확하게 구분할 수 있도록 수정.

JWT 검증 및 사용자 인증 로직을 분리하여 JwtTokenVerifier를 활용하도록 변경. Optional을 사용하여 토큰 추출 및 검증을 간결하게 처리하고, 인증 객체 생성을 UserDetailsFactory로 위임하여 책임을 명확히 분리함.

[🔀 merge] [4차] 코드 리펙토링 사항 적용 (4)

- 로깅 내용 수정 (tracId도 찍히도록) - 로깅 파일 보관일 수정 (30일 > 14일)

[🔨 fix] Docker 컨테이너 로그로도 로그 확인할 수 있도록 HTTP 요청/응답 로깅 설정 수정

[🔀 merge] HTTP 요청/응답 수정사항 로깅 스테이징 서버 반영

- 스크랩 취소시 유저 취소 권한 여부를 먼저 확인 후 스크랩 수를 update하도록 메서드 호출 순서를 수정했습니다.

- 스크랩 데이터를 삭제하고, 스크랩 수를 update하는 것이 더 안전하게 데이터 정합성을 지킬 수 있다고 생각되어 순서를 변경했습니다.

…코드 작성

- 실제 에러가 발생했는지 에러메시지로 비교하는 과정 추가했습니다.

[🔨 fix] 스크랩 추가시, 스크랩 수에 대한 동시성 테스트 진행

[feat] 소셜로그인 시, 알림서버와 운영서버의 유저를 동기화 하는 로직 추가

[♻️ refactor/#237] 도메인 기반 폴더링 구조로 변경

[🔀 merge] [4차] 코드 리펙토링 사항 스테이징 서버에 적용 (6)

junggyo1020 and others added 30 commits January 19, 2025 02:48

Merge pull request #195 from teamterning/main

6b6e2f5

[🔥 !hotfix] [3차] 스테이징 서버 마이그레이션 진행

refactor(Jwt) : 패키지 경로 변경

3b11d0f

- `org.terning.terningserver.jwt` → `org.terning.terningserver.auth.jwt`로 변경 - 패키지 구조 정리로 가독성과 유지보수성 향상

refactor(SignUpFilterResponseDto) : 불필요한 DTO 제거

0ab01f5

- 사용되지 않는 `SignUpFilterResponseDto` 제거 - 코드 정리로 불필요한 클래스 제거하여 유지보수성 향상

refactor(directory) : 패키지 경로 변경 및 구조 정리

1870316

feat(AuthSignInService) : 로그인 서비스 인터페이스 구현

356be8c

- `AuthSignInService` 인터페이스 구현하여 로그인 로직 추상화 - `signIn` 메서드를 정의하여 사용자 로그인 기능 제공 - 인증 서비스 구조 개선을 위해 역할 분리

feat(AuthSignOutService) : 로그아웃 서비스 인터페이스 구현

b8c120e

- `AuthSignOutService` 인터페이스 구현하여 로그아웃 로직 추상화 - `signOut` 메서드를 정의하여 사용자 로그아웃 기능 제공 - 인증 서비스 구조 개선을 위해 역할 분리

merge(auth) : Auth application 계층 구조 개선

d0853a2

[♻️ refactor/#196] Auth application 계층 구조 개선

junggyo1020 and others added 24 commits April 19, 2025 23:17

[✨ feat] 회원가입 시, 푸시알림여부 Default값으로 유저를 생성하도록 로직 수정

b4a5b48

Merge pull request #233 from teamterning/develop

823b3e7

[🔀 merge] [4차] 코드 리펙토링 사항 적용 (4)

[FIX] 파일 로깅 설정 수정

08a763f

- 로깅 내용 수정 (tracId도 찍히도록) - 로깅 파일 보관일 수정 (30일 > 14일)

[🔨 fix] Docker 컨테이너 로그로도 로그 확인할 수 있도록 HTTP 요청/응답 로깅 설정 수정

55b4e14

[🔨 fix] Docker 컨테이너 로그로도 로그 확인할 수 있도록 HTTP 요청/응답 로깅 설정 수정

[🔀 merge] HTTP 요청/응답 수정사항 로깅 스테이징 서버 반영

8cc1310

[🔀 merge] HTTP 요청/응답 수정사항 로깅 스테이징 서버 반영

[TEST] 스크랩 추가시에 대한 동시성 테스트코드 작성

3372e62

[REFACTOR] 스크랩 취소시 유저 권한 후 스크랩 수 update 하도록 호출 수정

3a4f3eb

- 스크랩 취소시 유저 취소 권한 여부를 먼저 확인 후 스크랩 수를 update하도록 메서드 호출 순서를 수정했습니다.

[REFACTOR] 스크랩 수 update 후 스크랩 저장하도록 순서 변경

91d0c34

[REFACTOR] ScrapServiceImpl 불필요한 개행 제거

75a5cc1

[FIX] 스크랩 취소 시, 스크랩 삭제 후 count 감소하도록 순서 수정

211368f

- 스크랩 데이터를 삭제하고, 스크랩 수를 update하는 것이 더 안전하게 데이터 정합성을 지킬 수 있다고 생각되어 순서를 변경했습니다.

[FEAT] 스크랩 취소시에 Unchecked Exception 발생 시 트랜잭션 롤백이 정상적으로 처리되는지에 대한 테스트…

965692f

…코드 작성

[FIX] 스크랩 취소시에 Unchecked Exception 발생 시에 대한 테스트 코드 수정

152111f

- 실제 에러가 발생했는지 에러메시지로 비교하는 과정 추가했습니다.

[🔨 fix] 스크랩 추가시, 스크랩 수에 대한 동시성 테스트 진행

50f2d06

[🔨 fix] 스크랩 추가시, 스크랩 수에 대한 동시성 테스트 진행

[feat] 소셜로그인 시, 알림서버와 운영서버의 유저를 동기화 하는 로직 추가

49a1f27

Merge remote-tracking branch 'origin/develop' into develop

a45fa6b

[feat] 소셜로그인 시, 알림서버와 운영서버의 유저를 동기화 하는 로직 추가

20544c2

Merge pull request #239 from teamterning/feat/#238

3368b69

[feat] 소셜로그인 시, 알림서버와 운영서버의 유저를 동기화 하는 로직 추가

Merge remote-tracking branch 'origin/develop' into develop1

176db6a

[refactor] 도메인 기반 폴더링 구조로 수정 - common

e70d7be

[refactor] 도메인 기반 폴더링 구조로 수정 - common

1b51795

[refactor] 도메인 기반 폴더링 구조로 수정

34ea30b

[refactor] 도메인 기반 폴더링 구조로 수정

9ea6df6

Merge pull request #240 from teamterning/refactor/#237

5c927be

[♻️ refactor/#237] 도메인 기반 폴더링 구조로 변경

Merge pull request #241 from teamterning/develop

c2d80e0

[🔀 merge] [4차] 코드 리펙토링 사항 스테이징 서버에 적용 (6)

junggyo1020 self-assigned this May 11, 2025

junggyo1020 added 🔀 merge 다른 브랜치와 병합 🐶정교🐶 🎉deploy CI/CD 관련 코드 수정 및 추가 labels May 11, 2025

pull-request-size bot added the size/XXL label May 11, 2025

junggyo1020 merged commit fc678dc into main May 11, 2025
4 of 6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[🔀 merge] [4차] 코드 리펙토링 사항 메인 서버에 적용 #242

[🔀 merge] [4차] 코드 리펙토링 사항 메인 서버에 적용 #242

Uh oh!

junggyo1020 commented May 11, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

[🔀 merge] [4차] 코드 리펙토링 사항 메인 서버에 적용 #242

[🔀 merge] [4차] 코드 리펙토링 사항 메인 서버에 적용 #242

Uh oh!

Conversation

junggyo1020 commented May 11, 2025

📄 Work Description

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants