-
Notifications
You must be signed in to change notification settings - Fork 1
test: ci/cd 테스트 #58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
test: ci/cd 테스트 #58
Conversation
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
- exceptionHandling에 authenticationEntryPoint 추가 - authenticationFilter에서 예외를 더 구체적으로 처리하도록 수정 - refresh token이 없는 채로 요청시 UNAUTHORIZED가 아닌 BAD_REQUEST를 반환하도록 변경
- 시간이 없어 정확한 원인은 파악한 뒤 이슈에 따로 남기고 언급 하겠습니다..
refactor: 인증과정 예외처리 #55
* feat: 수강신청 목록 초기화 api 구현 * refactor: 수강 신청 과목 제외하고 강좌 목록 반환
- 원래 관심과목 목록도 제외했었는데 수강신청된 목록만 제외하도록 변경하였습니다.
* fix: 수강 신청 동시성 이슈 해결 - CourseRegistration 엔티티에 유니크 제약 조건 추가 - registerCourse 메서드에서 중복 체크 로직 개선 * refactor: 예외처리 수정 - 상황에 맞지 않는 익셉션을 사용하던 것 수정 - BadRequestException 추가
* feat: 관심과목 담기 동일한 학수번호 예외처리 및 리팩토링 - 관심과목 목록에 동일한 학수번호로 신청된 정보가 존재하면 예외처리를 하였습니다. - body에 등록할 과목을 리스트로 받던 것을 과목 하나씩만 입력받도록 변경했습니다. * feat: 수강신청 동일한 학수번호 예외 처리 - 동일한 학수번호로 수강신청된 과목이 있는 경우도 예외처리 하였습니다.
* refactor: 매크로 코드 업데이트 * refactor: random 변수 수정
* refactor(TS-16): 프로젝트 구조 개선 - common과 domain 패키지로 구조 재분류 * refactor(TS-16): JWT 관련 파일을 common/security 패키지로 이동 - JWT 인증 관련 파일들을 common/security 패키지로 통합 - 보안 로직의 중앙화 및 재사용성 개선 * refactor(TS-18): 예외 처리 로직 중앙화 및 표준화 - BaseException과 ErrorCode, ErrorResponse 도입으로 예외 처리 구조화 - GlobalExceptionHandler를 ResponseEntityExceptionHandler 확장하도록 변경 - AlreadyRegisteredException 리팩토링 * refactor(TS-18): ErrorCode 인터페이스 도입 및 도메인별 에러 코드 구현 - ErrorCode 인터페이스 생성으로 에러 코드 구조 표준화 - 도메인별 ErrorCode enum 구현 (예: GlobalErrorCode, CourseRegistrationErrorCode) * refactor(TS-18): wishlist, schedule, user 패키지의 예외 처리 개선 - 도메인별 커스텀 예외 도입 및 적용 - BusinessException을 상속한 필요 예외 클래스 구현 * chore(TS-18): build.gradle 수정
- JWT 관련 예외를 집중 처리하는 JwtExceptionFilter 구현 - Spring Security 인증 예외를 관리하는 JwtAuthenticationEntryPoint 구현 - 세분화된 오류 보고를 위해 SecurityErrorCode enum 확장 - 새로운 예외 처리 컴포넌트를 적용하도록 SecurityConfig 수정
* feat: Logback 설정 추가 및 기본 로깅 구성 완료 * feat: Logback 설정 추가 및 기본 로깅 구성 완료
* feat: Logback 설정 추가 및 기본 로깅 구성 완료 * feat: Logback 설정 추가 및 기본 로깅 구성 완료 * chore: submodule Update
* feat: Logback 설정 추가 및 기본 로깅 구성 완료 * feat: Logback 설정 추가 및 기본 로깅 구성 완료 * chore: submodule Update * chore: github action 코드 수정
* chore: SimpleCaptcha 라이브러리 의존성 추가 * refactor: Captcha 구현을 SimpleCaptcha로 변경 및 응답 형식 개선 - 기존 Captcha 구현을 SimpleCaptcha 라이브러리로 대체 - Captcha 이미지 응답 방식을 파일 경로에서 Base64 인코딩된 이미지로 변경 - 이미지 생성을 위한 CaptchaConfig 설정 추가 - 메모리 효율성 개선 (이미지 파일 저장/관리 불필요)
* feat: Schedule 엔티티에 관심과목 수 관리 기능 추가 - wish_count 필드 추가 (기본값 0, null 불가) - 관심과목 증가/감소 메서드 추가 (incrementWishCount, decrementWishCount) - decrementWishCount 메서드에 음수 방지 로직 포함 - lombok 어노테이션 활용하여 코드 개선 * feat: 관심과목 등록/삭제 시 wishCount 업데이트 로직 추가 - 관심과목 등록 시 해당 과목의 wishCount 증가 - 관심과목 삭제 시 해당 과목의 wishCount 감소 * feat: 인기 과목 조회를 위한 Repository 메서드 추가 - wish_count 기준 내림차순 정렬 쿼리 메서드 추가 - Pageable을 통한 조회 개수 제한 지원 * feat: 인기 과목 조회 API 구현 - 인기 과목 순위 계산 로직 구현 (동일 wish_count는 같은 순위 부여) - 인기 과목 조회 API 엔드포인트 추가 (/api/schedules/popular) - limit 파라미터를 통한 조회 개수 제어 기능 추가 * refactor: 인기 과목 순위 계산 로직 개선 - 순위 계산 로직을 RankInfo 클래스로 분리하여 캡슐화 - calculatePopularSchedulesWithRank 메서드 추출로 책임 분리
- SecurityConfig의 하드코딩된 CORS 설정을 외부 설정 파일로 이동 - allowed-origins와 allow-credentials 설정을 프로퍼티로 분리
* chore: swagger 초기 세팅 * chore: submodule 업데이트 * chore: swagger 로그인 설정 * chore: Auth 도메인 swagger 설정 * chore: registraion 도메인 swagger 설정 * chore: schedule 도메인 swagger 설정 * chore: wishlist 도메인 swagger 설정 * refactor: shouldNotFilter 메서드 수정 * refactor: 에러 응답 일관성 수정 * refactor: Swagger SecurityConfig 구조 수정 * refactor: Swagger 관련 컴포넌트 개선 - SwaggerAccessDeniedHandler를 common/security 패키지로 이동 - SwaggerAuthenticationEntryPoint에서 불필요한 setStatus 제거 (ErrorResponse에서 처리) - 불필요한 ServletException throws 제거 * chore: 파일 끝 개행 삭제 * chore: redundant parameter 제거 - required=false 제거 - 스프링독(SpringDoc) OpenAPI 문서의 @parameter 어노테이션에서 required 속성의 기본값이 이미 false이므로 불필요. * chore: 파일 끝 개행 추가 --------- Co-authored-by: zhy2on <[email protected]>
- SecurityErrorCode와 ScheduleErrorCode의 중복을 피하기 위해 SecurityErrorCode의 접두사를 Auth의 A로 바꿈 - GlobalErrorCode에 G006이 중복돼 있던 것 수정
- 토큰 만료 시 로깅을 위한 username 추출 방식 변경 - ExpiredJwtException에서 직접 username을 추출하도록 수정하여 무한 재귀 문제 해결
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.
No description provided.