Merged
Conversation
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: 예외 상황, 검증 실패
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.