chore/16 - fix: feign 코드 수정

[ji-circle]

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.

• feign 코드 수정, url 제거

🚀 주요 변경 사항

완료한 이슈 번호
Close #16
관련된 이슈 번호 (닫고 싶지 않은 경우)
Related to #

✅ 자체 체크리스트 (필수)

• ./gradlew build 실행 결과 정상 (인증샷 첨부)
• 팀 내 컨벤션 준수 및 불필요한 로그, import 제거
• 중요한 변경 사항이 팀에 공유되었는지

📸 테스트 인증샷

빌드 결과 및 IntelliJ HTTP Client 실행 화면을 여기에 첨부해 주세요.

💬 리뷰어 전달사항 (선택)

특별히 봐주었으면 하는 부분이나 논의가 필요한 점을 적어주세요.

---

📎 참고 자료

관련 문서, 레퍼런스 링크 등이 있다면 여기에 첨부해주세요.

Summary by CodeRabbit

• Chores
  • 마이크로서비스 간 통신 설정을 간소화하여 서비스 레지스트리 기반 디스커버리로 개선했습니다.
  • Kafka 컨슈머 설정을 최적화하여 기본 설정을 활용하도록 업데이트했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 30d5ae9a-b41f-4d90-a38f-3ea29cbf91f9

📥 Commits

Reviewing files that changed from the base of the PR and between 018516a and 2fe761b.

📒 Files selected for processing (3)

• src/main/java/com/michelet/order/infrastructure/client/CatalogClient.java
• src/main/java/com/michelet/order/infrastructure/client/ReservationClient.java
• src/main/resources/application-prod.yml

💤 Files with no reviewable changes (1)

• src/main/resources/application-prod.yml

---

📝 Walkthrough

워크스루

Feign 클라이언트 두 개의 명시적 URL 구성을 제거하고 production 환경의 Kafka 컨슈머 직렬화 설정을 단순화합니다.

변경 사항

구성 정리 및 서비스 디스커버리 전환

Layer / File(s)	요약
Feign 클라이언트 서비스 이름 해석 src/main/java/com/michelet/order/infrastructure/client/CatalogClient.java, src/main/java/com/michelet/order/infrastructure/client/ReservationClient.java	CatalogClient와 ReservationClient에서 @FeignClient 어노테이션의 url 속성을 제거하고 서비스 이름 기반 해석으로 전환합니다.
Kafka 컨슈머 직렬화 설정 정리 src/main/resources/application-prod.yml	production 환경 설정에서 spring.kafka.consumer 하위의 key-deserializer와 value-deserializer 설정을 제거하고 group-id만 유지합니다.

🎯 2 (Simple) | ⏱️ ~8분

제안 검토자

• Jinyoung-Kim96
• qldo
• Sehi55

시

🐰 하드코딩을 치우니 깔끔하네요,
설정값들이 정리되어 마음도 편해요,
서비스 이름으로 우아하게 통신하고,
기본값의 미학을 신뢰하니까요!

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	PR이 #16 이슈의 'feign 하드코딩 제거' 주요 목표는 충족하였으나, 이슈의 체크리스트 항목(Dockerfile 검증, TODO/FIXME 해결, Request DTO 검증, 예외 클래스 분리)에 대한 코드 변경이 없습니다.	Dockerfile 검증, TODO/FIXME 주석 해결, Request DTO 검증, 도메인 패키지 내 예외 클래스 분리 등 체크리스트의 나머지 항목들을 완료하고 관련 코드 변경을 추가하세요.
Out of Scope Changes check	❓ Inconclusive	Kafka consumer 설정 변경(deserializer 제거)은 #16 이슈에서 명시된 범위(feign 하드코딩 제거)를 벗어난 변경으로 보입니다.	application-prod.yml의 Kafka 설정 변경이 #16 이슈와의 관련성을 명확히 하거나, 별도의 이슈로 추적할 필요가 있습니다.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 'feign 코드 수정'이라고 하였으나, 실제 변경 사항은 Feign 클라이언트에서 hardcoded URL 제거, Kafka 설정 변경 등 여러 부분을 포함하고 있어 제목이 주요 변경 사항을 완전히 반영하지 못했습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/16-feign-remove-url

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}