Conversation
# Conflicts: # src/main/java/inu/codin/codin/domain/block/entity/BlockEntity.java # src/main/java/inu/codin/codin/domain/email/controller/EmailController.java # src/main/java/inu/codin/codin/domain/email/service/EmailAuthService.java # src/main/java/inu/codin/codin/domain/lecture/domain/review/controller/ReviewController.java # src/main/java/inu/codin/codin/domain/report/service/ReportService.java # src/main/java/inu/codin/codin/domain/user/entity/UserEntity.java
- 상위 도메인 Repository 직접 참조 제거 (Comment → Post 등) - 상위 도메인 쿼리 서비스(PostQueryService 등)에서 유효성 검사 처리 - 의존성 순환 문제 발생 → Query/Command 분리로 해결
- Post 도메인 내 Scheduler 및 ErrorCode 의존성 정리 - 도메인 계층별로 올바른 ErrorCode 및 예외 처리 방식 적용
- 각 서비스내 redisBestService 호출 Best Domain으로 책임 이동
…헤더로-변경 226 fix access 인증 과정을 쿠키에서 인증 헤더로 변경
…헤더로-변경 226 fix access 인증 과정을 쿠키에서 인증 헤더로 변경
미사용해서 제거합니다. ㅎㅎ
…좋아요-반영 230 feat 외부에서 호출하는 lecture, review의 좋아요 반영
전반적인 refactoring 진행한 내역
lecture 서버에서 사용할 like feign 로직 추가 및 리팩토링
- Event 엔티티/DTO 수령시간 관련 필드 삭제 - 권한 검증 로직에서 ROLE_ 접두사 누락 문제 해결 (SecurityUtil.hasRole) - 이벤트 날짜/시간 응답 포맷 한글로 변경 - 이벤트 응답 DTO에 Campus 값 누락 문제 수정 - validationEvent 로직 조건 수정 (MANAGER 권한 추가) - 엑셀 내보내기에서 상품 수령 완료 유저만 포함되도록 조건 추가 - 유저 티켓팅 정보 검증 조건에 name null/길이 체크 추가 - 티켓팅 비밀번호 JSON 파싱 시 number 처리 문제 수정 - EventControllerTest 리팩토링
- Elasticsearch Java Client 8.16.3 마이그레이션 (High-Level REST Client → ES Java Client) - LectureElasticRepositoryImpl 새로 추가 (ES Java Client 기반) - LectureIndexMapping 인덱스 매핑 정의 클래스 추가 - ElasticsearchConfig, BulkIngestListenerImpl 설정 추가 - LectureStartupIndexer 리팩토링 - LectureElasticService 간소화 - ErrorCode enum에 logEvent() (log level) 추가 - GlobalErrorCode 인터페이스에 logEvent() default 메서드 추가 - logback-spring.xml, logstash-logback-encoder 의존성 추가 - application.yml ES 설정 변경 (인증/SSL 지원)
— 각 서비스 레포(core, auth, ticketing, lecture) 최신 코드를 모노레포로 통합 — UserRole을 codin-core에서 codin-security 모듈로 이동 — .gitmodules 삭제 (서브모듈 방식 폐기) — 테스트 의존성 H2 추가 (lecture, ticketing) — 이메일 템플릿, 프로필별 설정, 테스트 리소스 추가
— 4개 서비스별 GitHub Actions 워크플로우 구성 (core, auth, ticketing, lecture) — path filter 기반 트리거로 변경된 서비스만 빌드/배포 — 통합 docker-compose.yml 생성 (5개 서비스 + 외부 네트워크) — docker-compose.yml 변경 시 자동 서버 동기화 (SCP) — codin-core Dockerfile jar 이름 수정, codin-auth Dockerfile 신규 생성 — codin-lecture-api entrypoint.sh 추가 (ES 인증서 import) — .env.example 포트 매핑을 nginx 설정에 맞춰 정렬 — codin-core build.gradle에서 삭제된 서브모듈 참조 제거
- ticketing 워크플로우를 api/sse 개별 파이프라인으로 분리 - docker-compose.yml을 docker/prod/로 이동 및 경로 반영 - SSH 인증 방식 통일 (passphrase/password) - 각 모듈 application-prod.yml에 permit-all URL 설정 추가 - ticketing-sse 테스트용 application.yml 추가
- codin-core MongoDB 설정에 username/password/authSource 추가 - lecture-api 포트 매핑 8085:8080 → 8085:8085로 1:1 통일 - .env.example에 MongoDB 인증 변수 및 lecture 포트 반영
- feign.user-service.url을 yml에 명시 (USER_CLIENT_URL 환경변수 사용) - Docker 환경에서 codin-core-api로 통신 가능하도록 설정 - 테스트 의존성 추가 (spring-boot-starter-test, security-test)
— bbb8421 커밋에서 추가된 username/password/authentication-database 설정이 원래 멀티레포(CodIN-INU/BACKEND)에는 존재하지 않았음 — 해당 환경변수 미설정 시 char[] 바인딩 실패로 로컬 실행 불가 원인
— 로컬 ES는 xpack.security.enabled=false (TLS 없음)이므로 기존 ElasticsearchConfig의 CA 인증서 필수 로직으로 로컬 실행 불가 — prod 프로필: 기존 TLS+CA 인증서 설정 유지 — local 프로필: TLS 없이 http 연결하는 ElasticsearchLocalConfig 추가
— @PreAuthorize 등 메서드 수준 보안 어노테이션이 동작하도록 활성화
— 모노레포 내 다른 모듈과 구분하기 위해 메인 클래스명 명확화
— OAuth2LoginSuccessHandler, Apple/Google AuthService, CustomOAuth2UserService 테스트
— src/main/resources/data/ 에 개인정보 포함된 JSON 덤프가 있어 추적 제외
— 서버 MongoDB는 인증 필요하므로 application-prod.yml에 추가 — profiles.active를 local만 남겨 로컬에서는 prod.yml 미로드 — 서버는 SPRING_PROFILES_ACTIVE=prod로 오버라이드
— main, dev 브랜치 PR 시 전체 모듈 빌드 + 테스트 실행 — codin-core Dockerfile: trailing backslash로 인한 빌드 실패 수정
— CD와 동일한 path 필터로 변경된 모듈만 빌드+테스트 — main, dev 브랜치 PR 대상
— docker-compose에서 실제 사용하는 포트와 일치시킴
chore: 멀티레포 → 모노레포 전환 및 CI/CD 파이프라인 구축
Contributor
|
Important Review skippedToo many files! This PR contains 300 files, which is 150 over the limit of 150. You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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
3 participants
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.