Skip to content

Conversation

@tape4
Copy link
Member

@tape4 tape4 commented Oct 11, 2025

No description provided.

tape4 added 4 commits October 11, 2025 10:26
Member 도메인:
- MemberValidationService 생성하여 중복 검증 로직 중앙화
- Controller에 Swagger 어노테이션 추가 (ApiResponses 제외)
- Repository 메소드 JPQL 쿼리 최적화

Font & Bookmark 도메인:
- FontValidationService 생성하여 검증 로직 통합
- Controller에 Swagger 어노테이션 추가
- Repository 쿼리 최적화 및 existsByMemberIdAndName 추가

Provide, File, SMS 도메인:
- JPQL 쿼리 최적화
- Native Query 적용

비도메인 부분:
- JwtTokenProviderImpl: 중복 코드 제거 및 헬퍼 메소드 추가
- GlobalExceptionHandler: HTTP 상태별 예외 그룹화, 제네릭 핸들러 제거
- DiscordExceptionLoggingAspect: 코드 간소화
- SecurityConfig: AbstractHttpConfigurer 사용
서비스 레이어:
- MemberCreationServiceImpl: 회원 생성 로직 주석 추가
- FontServiceImpl: 핵심 비즈니스 로직 메서드별 상세 주석 추가

컨트롤러:
- FontController: REST API 엔드포인트 설명 추가

레포지토리:
- FontRepositoryImpl: 데이터 접근 계층 메서드 설명 추가

설정 클래스:
- WebConfig: CORS, Interceptor, ArgumentResolver 설정 주석 추가

보안/인증:
- JwtAuthenticationFilter: JWT 토큰 처리 로직 상세 주석 추가
서비스 레이어:
- AuthService: 토큰 발급 로직 주석 추가
- BookmarkServiceImpl: 북마크 생성/삭제/조회 로직 주석 추가
- MemberLookupServiceImpl: 회원 조회 로직 주석 추가
- MemberOnboardServiceImpl: OAuth2 온보딩 프로세스 주석 추가
- MemberUpdateServiceImpl: 회원 정보 수정 및 비활성화 로직 주석 추가
- ProvideServiceImpl: OAuth2 제공자 정보 관리 로직 주석 추가

컨트롤러:
- MemberController: 회원 정보 조회 API 주석 추가
서비스별 로깅 개선:
- MemberCreationServiceImpl: 회원 생성 프로세스 로깅
- MemberLookupServiceImpl: 회원 조회 및 검증 로깅
- MemberOnboardServiceImpl: OAuth2 온보딩 프로세스 로깅
- MemberUpdateServiceImpl: 회원 정보 수정 및 비활성화 로깅
- ProvideServiceImpl: OAuth2 제공자 정보 관리 로깅
- BookmarkServiceImpl: 북마크 생성/삭제/조회 로깅

로깅 레벨:
- INFO: 비즈니스 로직 진입/완료, 중요 상태 변경
- DEBUG: 상세 처리 과정, 중간 단계 정보
- WARN: 예외 상황, 검증 실패
@tape4 tape4 requested a review from SiwonHae as a code owner October 11, 2025 02:51
@tape4 tape4 merged commit a3f246a into develop Oct 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants