🔀 :: [#723] - 도메인 연결시 설정파일이 생성되어도 트래픽이 전달되지 않는 현상 수정

[dolong2]

개요

• 도메인을 연결후 성공적으로 연결되었고, 파일이 반영되었지만, nginx가 리버스 프록싱을 못하는 현상을 수정합니다.

작업내용

• http 설정시 애플리케이션의 containerName을 통한 경로로 프록싱하도록 수정

체크리스트

탬플릿외에 필요한 항목이 있으면 추가해주세요.

• 로컬에서 빌드가 성공하나요?
• 추가(수정)한 코드가 정상적으로 동작하나요?
• pr 타켓 브랜치가 맞게 설정되어 있나요?
• pr에서 작업할 내용만 작업됐나요?
• 기존 API와 호환되지 않는 사항이 있나요?

Summary by CodeRabbit

• 버그 수정
  • Docker 환경에서 프록시 라우팅 불안정으로 인한 간헐적 연결 실패를 해소했습니다. 요청 전달의 안정성과 응답 시간 일관성이 개선되고, 재시도/타임아웃 발생률이 감소했습니다.
• 작업(Chores)
  • 컨테이너화된 배포에 맞춰 HTTP 프록시 구성을 정비했습니다. 로컬·CI·운영 환경 간 동작 일관성이 향상되고, 네트워크 해상도 의존성이 줄어 설정 관리가 단순해졌습니다.

[coderabbitai]

Walkthrough

Nginx 애플리케이션 HTTP 설정을 생성하는 코드에서 proxy_pass 대상이 host.docker.internal에서 애플리케이션 컨테이너 이름과 포트로 변경되었습니다. 다른 로직 변경이나 공개 API 변경은 없습니다.

Changes

Cohort / File(s)	Change Summary
Nginx 프록시 대상 업데이트 src/main/kotlin/.../common/file/FileContent.kt	getApplicationHttpConfig 내 Nginx 스니펫에서 proxy_pass를 http://host.docker.internal:${application.externalPort}에서 http://${application.containerName}:${application.externalPort}로 변경

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Client as 클라이언트
  participant Nginx as Nginx(앱 프록시)
  participant Backend as 애플리케이션 컨테이너

  rect rgb(240,245,255)
  note over Nginx: 변경점
  Client->>Nginx: HTTP 요청
  Nginx->>Backend: proxy_pass http://${application.containerName}:${application.externalPort}
  Backend-->>Nginx: 응답
  Nginx-->>Client: 최종 응답
  end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• 🔀 :: [#626] - 도메인 연결시 생성되는 Nginx 파일 수정 #627: 동일 함수(getApplicationHttpConfig)에서 proxy_pass 대상을 수정한 PR로, 이 변경과 직접 충돌/연속 가능성이 큼.
• 🔀 :: [#668] - Nginx 설정 수정 #669: 같은 Nginx 설정 생성 로직에서 server 블록 문법을 다룬 수정으로, 본 변경과 접점 있는 부분을 만짐.
• 🔀 :: [#577] - nginx 설정 파일 수정 #578: proxy_pass를 host.docker.internal로 바꿨던 변경으로, 이번 PR이 이를 컨테이너 이름으로 되돌리는 맥락과 연관.

Poem

컨테이너 이름을 톡톡, 귀로 두드려 보니
길 잃던 패킷도 토끼굴을 바로 찾네 🐇
host를 떠나, 이름 따라 점프!
프록시의 달빛 길 위에
응답은 살포시, 포트에 앉는다 ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	제목은 도메인 연결 후 설정 파일이 생성되어도 nginx가 트래픽을 전달하지 못하는 문제를 정확히 요약하여 변경사항과 일치하며 협업 시 이력을 스캔하는 동료가 주된 수정 내용을 파악하기에 충분히 구체적입니다.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/domain-connect-logic

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2a338b3 and 484ce3c.

📒 Files selected for processing (1)

• src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build and Test

🔇 Additional comments (1)

src/main/kotlin/com/dcd/server/core/common/file/FileContent.kt (1)

121-121: nginx 컨테이너도 동일 네트워크에 연결하도록 배포 설정 확인
Application 모델에서 containerName이 ${name.replace(" ", "_").lowercase()}-$id로 고유하게 생성되며,
CreateContainerServiceImpl에서 docker create --network ${application.workspace.networkName}와
docker network connect 명령어로 애플리케이션 컨테이너를 네트워크에 연결하고 있습니다.
따라서 nginx 컨테이너가 해당 워크스페이스 네트워크에 조인되어 있는지 운영 환경 설정만 검증해 주세요.

---

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

🧪 Early access (Sonnet 4.5): enabled

We are currently testing the Sonnet 4.5 model, which is expected to improve code review quality. However, this model may lead to increased noise levels in the review comments. Please disable the early access features if the noise level causes any inconvenience.

Note:

• Public repositories are always opted into early access features.
• You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file.

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