Skip to content

HOTFIX : nginx 설정 변경#196

Merged
alsgud8311 merged 1 commit intomainfrom
development
Nov 27, 2025
Merged

HOTFIX : nginx 설정 변경#196
alsgud8311 merged 1 commit intomainfrom
development

Conversation

@alsgud8311
Copy link
Contributor

@alsgud8311 alsgud8311 commented Nov 27, 2025

HOTFIX : prod 클라이언트 서버에 대한 nginx 설정 변경

@alsgud8311 alsgud8311 merged commit f2141b9 into main Nov 27, 2025
6 checks passed
@gemini-code-assist
Copy link
Contributor

gemini-code-assist bot commented Nov 27, 2025

Summary of Changes

Hello @alsgud8311, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 프로덕션 환경의 Nginx 설정을 최적화하고 개선하기 위한 핫픽스입니다. 주요 변경 사항은 WebSocket 지원을 위한 설정 추가와 HTTP/2 프로토콜 활성화 방식 개선이며, 전반적인 설정 파일의 가독성과 정확성을 높이는 주석 업데이트도 포함됩니다. 이는 클라이언트 서버의 안정성과 성능 향상에 기여할 것입니다.

Highlights

  • WebSocket 지원을 위한 Nginx 설정 추가: HTTP 업그레이드 헤더를 기반으로 connection_upgrade 변수를 매핑하는 map 지시어를 추가하여 WebSocket 연결을 올바르게 처리하도록 설정했습니다.
  • HTTP/2 설정 개선: listen 지시어에서 http2 키워드를 분리하고 http2 on;을 명시적으로 추가하여 deprecated 경고를 해결하고 HTTP/2 프로토콜을 명확히 활성화했습니다.
  • 주석 및 설명 업데이트: Nginx 설정 파일 내의 여러 주석을 최신화하고 명확성을 높였습니다. 예를 들어, dev.kokomen.kr 관련 주석을 kokomen.kr로 변경하고, 타임아웃/버퍼 설명에서 '빌드' 부분을 제거했습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

gemini-code-assist[bot]
gemini-code-assist bot reviewed Nov 27, 2025
View reviewed changes
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

이 PR은 운영 환경의 Nginx 설정을 수정하는 긴급 수정(hotfix)입니다. WebSocket 지원을 개선하고 오래된(deprecated) 지시어를 수정하는 등 코드 변경 자체는 올바릅니다. 하지만, 수정된 파일이 실제 운영 환경에서 사용되지 않을 수 있는 심각한 문제를 발견했습니다. docker/server/compose.prod.yaml 설정에 따르면 다른 경로의 설정 파일이 사용되고 있는 것으로 보입니다. 이 부분이 해결되지 않으면 hotfix가 적용되지 않으므로 반드시 확인이 필요합니다. 자세한 내용은 코드 리뷰 코멘트를 참고해주세요. 추가적으로, 수정된 설정 파일 내에 로그 파일 경로가 dev 환경으로 되어있는 부분도 확인해보시면 좋을 것 같습니다.

nginx/nginx.prod.conf
Comment on lines +14 to +17
map $http_upgrade $connection_upgrade {
default upgrade;
'' keep-alive;
}
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot Nov 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

critical

이 설정 파일(nginx/nginx.prod.conf)을 수정하고 계신 것으로 보입니다. 하지만 함께 제공된 docker/server/compose.prod.yaml 파일을 확인해보니, 실제 운영 환경의 nginx 컨테이너는 ../../apps/kokomen-server/nginx/prod/nginx.conf 파일을 마운트하여 사용하고 있습니다. 현재 수정 중인 파일이 실제 운영 환경에 적용되지 않을 수 있으니, 올바른 파일을 수정하고 있는지 반드시 확인해 보시기 바랍니다. 이는 긴급 수정(hotfix)의 의도와 다르게 변경 사항이 전혀 적용되지 않을 수 있는 심각한 문제입니다.

@github-actions
Copy link

github-actions bot commented Nov 27, 2025

🛠️ Build Summary

Status: ✅ SUCCESS
Duration: 62초
Exit Code: 0
Commit: 91bdd8b

📋 Build Output (마지막 45줄)

├ ○ /404 (1450 ms)                         1.77 kB         315 kB
├ ○ /500 (1447 ms)                           758 B         311 kB
├ ƒ /api/auth/logout                           0 B         310 kB
├ ƒ /dashboard                             14.3 kB         637 kB
├ ƒ /interviews                            8.19 kB         349 kB
├ ƒ /interviews/[interviewId]              11.7 kB         607 kB
├ ƒ /interviews/[interviewId]/result       7.78 kB         365 kB
├ ○ /layout (1448 ms)                        540 B         311 kB
├   └ css/2af4d3721e97fa9b.css               280 B
├ ƒ /login                                 2.59 kB         315 kB
├ ƒ /login/callback                        2.01 kB         336 kB
├ ƒ /login/google/callback                 2.02 kB         336 kB
├ ƒ /login/profile                         7.65 kB         369 kB
├ ƒ /members/[memberId]                       4 kB         341 kB
├ ƒ /members/[memberId]/sitemap.xml          429 B         311 kB
├ ƒ /members/interviews/[interviewId]      6.91 kB         341 kB
├ ƒ /purchase                              7.64 kB         631 kB
├ ƒ /purchase/confirm                      1.78 kB         335 kB
├ ○ /purchase/error (1447 ms)                800 B         313 kB
├ ƒ /rank                                  3.34 kB         340 kB
├ ƒ /recruit                               7.39 kB         348 kB
├ ƒ /resume                                  865 B         438 kB
├ ƒ /resume/eval                            2.4 kB         440 kB
├ ƒ /resume/eval/demo                      2.67 kB         440 kB
├ ƒ /server-sitemap.xml                      422 B         311 kB
├ ƒ /sitemap/members.xml                     424 B         311 kB
├ ƒ /sitemap/rank.xml                        421 B         311 kB
├ ƒ /terms/privacy                         5.23 kB         339 kB
└ ƒ /terms/termsofuse                      5.06 kB         339 kB
+ First Load JS shared by all               326 kB
  ├ chunks/framework-dc17cf7945e40527.js   57.6 kB
  ├ chunks/main-85275f8b1d68389c.js         176 kB
  ├ chunks/pages/_app-6579d1d69d18093d.js  74.7 kB
  ├ css/8582497d0c87733e.css               15.9 kB
  └ other shared chunks (total)            1.92 kB

ƒ Middleware                               96.2 kB

○  (Static)   prerendered as static content
ƒ  (Dynamic)  server-rendered on demand

   Memory usage report:
    - Total time spent in GC: 397.31ms
    - Peak heap usage: 69.27 MB
    - Peak RSS usage: 682.40 MB

🤖 Generated by GitHub Actions at Thu Nov 27 10:50:06 UTC 2025

@github-actions
Copy link

github-actions bot commented Nov 27, 2025

🚀 Lighthouse Report for TEST1

📅 Date: 11/27/2025

Category Score
🔴 Performance 16
🟢 Accessibility 96
🟢 Best Practices 96
🟢 SEO 100

📊 Performance Details

Metric Score Value
🟢 First Contentful Paint 97 1.4 s
🔴 Largest Contentful Paint 3 7.9 s
🔴 Cumulative Layout Shift 20 0.447

🚀 Lighthouse Report for TEST2

📅 Date: 11/27/2025

Category Score
🟠 Performance 56
🟢 Accessibility 96
🟢 Best Practices 96
🟢 SEO 100

📊 Performance Details

Metric Score Value
🟢 First Contentful Paint 100 0.9 s
🟠 Largest Contentful Paint 53 3.9 s
🟢 Cumulative Layout Shift 100 0

🚀 Lighthouse Report for TEST3

📅 Date: 11/27/2025

Category Score
🟠 Performance 51
🟢 Accessibility 96
🟢 Best Practices 96
🟢 SEO 100

📊 Performance Details

Metric Score Value
🟢 First Contentful Paint 100 0.9 s
🔴 Largest Contentful Paint 17 5.6 s
🟢 Cumulative Layout Shift 100 0

🚀 Lighthouse Report for TEST4

📅 Date: 11/27/2025

Category Score
🔴 Performance 42
🟢 Accessibility 96
🟢 Best Practices 96
🟢 SEO 100

📊 Performance Details

Metric Score Value
🟢 First Contentful Paint 100 0.9 s
🟠 Largest Contentful Paint 61 3.6 s
🔴 Cumulative Layout Shift 20 0.447

🚀 Lighthouse Report for TEST5

📅 Date: 11/27/2025

Category Score
🟠 Performance 59
🟢 Accessibility 96
🟢 Best Practices 96
🟢 SEO 100

📊 Performance Details

Metric Score Value
🟢 First Contentful Paint 100 0.9 s
🟠 Largest Contentful Paint 59 3.7 s
🟢 Cumulative Layout Shift 100 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@alsgud8311