Skip to content

[FIx] Multipart request size & 파라미터 수정#446

Merged
david-parkk merged 2 commits intodevelopfrom
fix/#445
Feb 27, 2026
Merged

[FIx] Multipart request size & 파라미터 수정#446
david-parkk merged 2 commits intodevelopfrom
fix/#445

Conversation

@david-parkk
Copy link
Contributor

@david-parkk david-parkk commented Feb 27, 2026

#️⃣ 연관된 이슈

closes #445

📝작업 내용

  • 파라미터 수정
  • 멀티파트 request size 변경

작업 상세 내용

상세 내용을 입력해주세요.

💬리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

Summary by CodeRabbit

Bug Fixes

  • 장소 정보의 콘텐츠 필드 데이터 수정

Chores

  • 파일 업로드 최대 크기를 10MB에서 50MB로 확대하여 더 큰 파일 업로드 지원
  • 구성 서브모듈 참조 업데이트

@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c0277ff and 229e286.

📒 Files selected for processing (3)
  • .github/workflows/backend-prod.yml
  • src/main/java/ku_rum/backend/domain/place/application/response/SelectPlaceChipResponse.java
  • src/main/resources/config

Walkthrough

이 PR은 세 가지 변경을 포함합니다: Nginx 요청 본문 크기 제한을 10M에서 50M으로 증가시키고, SelectPlaceChipResponse에서 콘텐츠 소스를 약자(abbreviation)에서 실제 콘텐츠(content)로 변경하며, 설정 서브모듈 참조를 업데이트합니다.

Changes

Cohort / File(s) Summary
배포 설정
.github/workflows/backend-prod.yml
Nginx HTTPS 서버 블록의 client_max_body_size를 10M에서 50M으로 증가하여 프록시된 요청의 최대 허용 본문 크기를 확대.
응답 필드 매핑
src/main/java/ku_rum/backend/domain/place/application/response/SelectPlaceChipResponse.java
두 개의 팩토리 메서드에서 place.getAbbreviation() 호출을 place.getContent()로 변경하여 응답의 content 필드에 실제 콘텐츠를 매핑.
서브모듈 업데이트
src/main/resources/config
설정 서브모듈 참조를 commit 20a8929에서 ad593df로 업데이트.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • [Fix] CD 변경 #310: 동일한 배포 워크플로우 파일(backend-prod.yml)의 Nginx client_max_body_size 설정을 수정하고 있어 관련 있음.
  • [Fix] 멀티파트 설정 변경 #416: 동일한 backend-prod.yml의 Nginx client_max_body_size 요청 본문 크기 제한을 변경하고 있어 관련 있음.

Suggested reviewers

  • kmw10693

Poem

🐰 미리보기에 콘텐츠가 빠졌다던데,
이제 요청도 크고 내용도 풍성하게!
약자는 물러나고 content가 나타났으니,
설정도 버전도 새로워져 반짝반짝✨
몸집도 늘었으니 더 많은 것을 담을 수 있겠네! 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 템플릿 구조를 따르고 있으나, '작업 상세 내용' 섹션이 '상세 내용을 입력해주세요'로만 기재되어 있어 구체적인 설명이 부족합니다. 각 변경사항(client_max_body_size 증가, getAbbreviation()에서 getContent()로 변경)에 대한 구체적인 이유와 영향을 작업 상세 내용에 추가해주세요.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Out of Scope Changes check ❓ Inconclusive 세 가지 변경사항 중 backend-prod.yml의 client_max_body_size 증가(10M→50M)는 이슈 #445의 content 누락 문제와 직접적인 연관이 없어 보입니다. backend-prod.yml의 client_max_body_size 변경이 이슈 #445 해결에 필요한 변경인지, 아니면 별도 작업인지 명확히 하기 위해 PR 설명을 보충해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 변경 사항의 주요 내용을 반영하고 있습니다. '멀티파트 request size & 파라미터 수정'은 백엔드 Nginx 설정 변경과 SelectPlaceChipResponse의 content 필드 수정을 명확하게 요약합니다.
Linked Issues check ✅ Passed PR의 코드 변경사항들이 연결된 이슈 #445 '미리보기시 content 누락' 요구사항을 충족합니다. SelectPlaceChipResponse에서 getAbbreviation()을 getContent()로 변경하여 content 누락 문제를 직접 해결합니다.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#445

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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

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

@sonarqubecloud
Copy link

@github-actions
Copy link

Test Results

 38 files   38 suites   9s ⏱️
173 tests 173 ✅ 0 💤 0 ❌
174 runs  174 ✅ 0 💤 0 ❌

Results for commit 229e286.

@david-parkk david-parkk merged commit a595174 into develop Feb 27, 2026
5 checks passed
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.

미리보기시 content 누락

1 participant