-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix : nginx max_body_size 변경 #103
Conversation
Walkthrough이번 변경 사항은 Nginx 설정 파일 내의 HTTP 블록에 새로운 구성 지시문 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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
infra/development/nginx/nginx.conf (2)
7-8
: 향후 개선 사항 고려현재는
http
블록에 글로벌하게client_max_body_size
를 설정했습니다. 이는 모든 요청에 적용되지만, 더 세밀한 제어를 위해 특정location
블록에만 이 설정을 적용하는 것도 고려해 볼 수 있습니다. 예를 들어, 파일 업로드를 처리하는 특정 API 경로에만 적용할 수 있습니다.http { client_max_body_size 50M; ... server { ... location / { proxy_pass http://api; proxy_set_header Host $http_host; proxy_set_header X-Real-IP $remote_addr; proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for; + # 파일 업로드를 처리하는 특정 경로에만 적용할 수도 있습니다 + # location /api/uploads { + # client_max_body_size 50M; + # proxy_pass http://api; + # ... + # } } } }
8-8
: 설정 변경 이유에 대한 주석 추가 권장이 설정 변경의 이유를 설명하는 주석을 추가하면 향후 유지보수에 도움이 될 것입니다. 이를 통해 다른 개발자들이 이 설정의 목적과 중요성을 쉽게 이해할 수 있습니다.
http { - client_max_body_size 50M; + # 413 Request Entity Too Large 오류 해결을 위해 최대 요청 바디 크기를 50M로 설정 + # 이슈 #102 참조: 파일 업로드 시 오류 발생 문제 해결 + client_max_body_size 50M; default_type application/octet-stream; ... }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
infra/development/nginx/nginx.conf
(1 hunks)
🔇 Additional comments (1)
infra/development/nginx/nginx.conf (1)
8-8
: 설정 변경이 올바르게 적용되었습니다.
client_max_body_size
설정을 50M로 변경한 것은 PR에서 언급된 "413 Request Entity Too Large" 오류를 해결하기 위한 적절한 접근 방식입니다. 이 설정을 통해 클라이언트가 더 큰 파일을 업로드할 수 있게 되었습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
#️⃣ 관련 이슈
💡 작업내용
파일 업로드 시
413 Request Entity Too Large
에러가 나 nginx 설정 수정하였습니다.📸 스크린샷(선택)
📝 기타