전환 후 프론트 경유 검증 추가 - /health 127.0.0.1 한정 개방 - #931
Conversation
- extractor 배포에는 있고 core 에는 없던 전환 후 검증(프론트 → nginx → 새 슬롯)을 통일한다. 이 검증이 잡는 것은 앱은 살아있는데 배달 경로가 끊긴 부류(extractor 이행 배포의 Host/444 사고가 실례)로, 중간 장치(TLS·레이트리밋·화이트리스트)가 더 많은 core 쪽이 오히려 검증 없이 배포되고 있었다 - nginx conf(dev·prod 동일): /health 를 박스 자신(127.0.0.1)에게만 열고 외부는 종전(#833)대로 444 유지. 외부 개방이 아니라 #833 의 재개방 조건과 별개다. 두 conf 에 함께 넣어 promote 때 prod 검증도 같이 성립한다 - deploy: slot_switch 에 verify-cmd 추가 - 블록이 이미 verify 를 원복 경계 안에서 지원(infra#39)하므로 인자 하나로 끝난다. curl -k 는 cert 가 도메인 앞이라 IP 접속의 hostname 검증만 끄는 것, --retry 는 reload 직후 워커 교체 흔들림 흡수 - 검증: 스텝 스크립트 bash -n·shellcheck 클린. conf 는 CI nginx-validate(동일 더미 우회 스크립트)가 게이트
- infra#40 이 --insecure·--host-header 를 지원해, 검증 배선(폴링·재시도·종료코드)을 블록 한 곳에 두고 호출부는 값만 준다 - extractor 의 verify 와 같은 모양
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
|
Warning Review limit reached
Next review available in: 59 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
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. Comment |
Situation
Task
Action
location = /health를 신설해 박스 자신(127.0.0.1)에게만 프록시하고, 그 외 출처는 종전 catch-all 과 같은 444 로 닫는다. 화이트리스트에서 /health 제거 - 외부 소비자 없음 실측 #833 의 "외부 uptime 모니터 도입 시 재개방" 조건과는 별개다 - 이건 외부 개방이 아니다. 두 conf 에 함께 넣어야 promote 때 prod 배포의 검증도 성립한다.--verify-cmd를 추가한다. 블록이 verify 를 원복 경계 안에서 이미 지원하므로(infra#39) 인자 하나로 끝난다 - 블록 추출의 이득이 회수되는 지점.--insecure는 cert 가 도메인 앞이라 IP(127.0.0.1) 접속의 hostname 검증만 끄는 것이고, SNI 미전송이어도 박스당 서버 블록이 하나뿐이라 default 매칭으로 닿는다. 짧은 재시도(2s x 5)는 reload 직후 워커 교체의 순간 흔들림 흡수.Result
연관 이슈