Skip to content

fix: route application.yml - #13

Merged
Sehi55 merged 1 commit into
devfrom
chore/12-edit-applicationyml
May 6, 2026
Merged

Sehi55 merged 1 commit into
devfrom
chore/12-edit-applicationyml

Conversation

@Sehi55

@Sehi55 Sehi55 commented May 6, 2026

Copy link
Copy Markdown
Contributor

📝 작업 내용

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

  • application.yml 수정

🚀 주요 변경 사항

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

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

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

📸 테스트 인증샷

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

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

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

  • 논의점


📎 참고 자료

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

Summary by CodeRabbit

변경 사항

  • 버그 수정
    • API 엔드포인트 HTTP 메서드 요청 제한 개선
    • 제품 관리 작업 라우팅 최적화

@Sehi55 Sehi55 self-assigned this May 6, 2026
@Sehi55 Sehi55 added the documentation Improvements or additions to documentation label May 6, 2026
@Sehi55 Sehi55 linked an issue May 6, 2026 that may be closed by this pull request
1 task
@coderabbitai

coderabbitai Bot commented May 6, 2026

Copy link
Copy Markdown

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: ccc88657-81a4-420f-ae1d-c14afa6c6ed5

📥 Commits

Reviewing files that changed from the base of the PR and between 7ac91e1 and a4c6738.

📒 Files selected for processing (1)
  • src/main/resources/application.yml

📝 Walkthrough

Walkthrough

Spring Cloud Gateway의 application.yml에서 경로와 HTTP 메서드 제약 조건을 추가했습니다. catalog-service는 GET 요청으로만 제한되고, inventory-service는 관리 경로를 포함하도록 확장되고 POST, PUT, PATCH, DELETE 메서드를 처리하도록 설정되었습니다.

Changes

Gateway 라우팅 설정 업데이트

Layer / File(s) Summary
Catalog 서비스 제약
src/main/resources/application.yml (32-36줄)
/api/v1/products/** 경로에 Method=GET 제약을 추가하여 읽기 전용으로 제한
Inventory 서비스 확장
src/main/resources/application.yml (40-41줄)
경로 제약을 /api/v1/products/**과 /api/v1/admin/products/**로 확장하고, Method=POST,PUT,PATCH,DELETE를 추가하여 쓰기 작업 처리
CORS 포맷팅
src/main/resources/application.yml (71줄)
allowed-origins 배열 포맷을 []에서 [ ]로 변경 (의미 변화 없음)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Miche-Let/api-gateway#6: catalog 및 inventory 라우트 설정에 대한 이전 변경사항과 관련된 동일한 파일 수정

Suggested reviewers

  • Jinyoung-Kim96
  • ji-circle
  • githyj-jang
  • jihxonx
  • qldo

Poem

🐰 게이트웨이를 다듬으며
GET은 열고, POST는 닫으니
마치 문지기 같은 설정이
요청들을 바른길로 인도하네 🚦
작지만 확실한 변화가
서비스의 경계를 그어준다네

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'fix: route application.yml' accurately describes the main changes, which involve updating route predicates in application.yml.
Linked Issues check ✅ Passed The PR directly addresses issue #12's objective to modify route information in application.yml by updating the catalog-service and inventory-service route predicates.
Out of Scope Changes check ✅ Passed The CORS formatting change is minor and tangential; all substantive changes focus on route predicate updates as specified in issue #12.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/12-edit-applicationyml

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

@Sehi55
Sehi55 merged commit 34255b6 into dev May 6, 2026
4 checks passed
@Sehi55
Sehi55 deleted the chore/12-edit-applicationyml branch May 6, 2026 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CHORE] application.yml 수정

1 participant