Skip to content

5주차 기준 코드 리뷰를 남깁니다.#197

Open
kimssy-gahyun wants to merge 797 commits intomainfrom
closetBuddy-MSA-dev
Open

5주차 기준 코드 리뷰를 남깁니다.#197
kimssy-gahyun wants to merge 797 commits intomainfrom
closetBuddy-MSA-dev

Conversation

@kimssy-gahyun
Copy link

  • 코드 리팩토링 관련
  1. User 도메인 외의 다른 도메인 분리
    지난 주 코드리뷰에서 언급되었던 것 처럼 MSA 구조 상 Cart, Product, Order 등의 다른 도메인들도 분리하고, 로직 상 필요한 부분이 있다면 FeignClient를 통해 백엔드 Controller 간의 요청, 응답으로 연결해 주시면 됩니다.
  • 프로젝트 구조 및 아키텍처
  1. 인증, 인가 공통기능 구현
    지난 번 언급되었던 인터셉터 방식 보다는 스프링 배치를 통해 자동화된 코드가 Controller 의 호출 없이도 실행될 점을 감안하여 내부 API 방식으로 구현한 점이 더 적합하다고 생각했습니다.

  2. Elastic Search 를 통한 검색 기능 구현
    Elastic Search가 JSON 구조를 기반으로 동작하는 점을 숙지하며 도큐먼트 구조에 맞는 쿼리를 작성해보시길 바랍니다.

  3. YOLO 모델을 통한 이미지 탐지 및 크롭 기능 추가
    왜 YOLO 모델을 이용하여 이미지 탐지를 하게 되었는지, 지도학습 관점과 모델 학습 정확도 개선 관점에서의 레이블 사용을 잘 정리해보시면 좋을 것 같습니다.

Machanyoung and others added 30 commits February 16, 2026 11:49
각 서비스의 test application.yml을 작성하여 업로드 합니다. 테스트용 파일이므로 보안에 영향이 없습니다.
- publish-common-event.yml branches 작성 방식을 다른 yml 에 도입

- 띄어쓰기 및 주석 변경
rksdu and others added 30 commits February 25, 2026 23:20
cors 오류를 해결하기 위한 Spring SecurityConfig의 cors 설정 적용
[#245] Oauth 로그인 경로는 인증 전 단계이므로 JWT 검증 필터를 건너 뛰도록 설정
같은 클래스 내 호출은 카운트 안 함
test 를 위한 의존성 추가
[#269] feign 호출 서비스 url 환경변수 주입
CI 테스트를 위한 test 코드 수정
[#246] - Redis Caching 처리 부분 수정, Swagger 수정, 에러 반환 코드 수정
[#271]cors 설정 변경으로 인한 프론트엔드 엔드포인트 통과 허용
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.

5 participants