Skip to content

배포의 TLS 인증서 존재 확인 수정 - #1233

Merged
m-a-king merged 2 commits into
devfrom
fix/deploy-cert-check
Oct 15, 2026
Merged

m-a-king merged 2 commits into
devfrom
fix/deploy-cert-check

Conversation

@m-a-king

Copy link
Copy Markdown
Collaborator

한눈에

배포 작업의 TLS 인증서 존재 확인 수정

  • 이전: 배포 작업이 일반 사용자 권한으로 인증서 폴더를 확인
    • 인증서 폴더는 root 만 열 수 있음
    • 결과: 인증서가 있어도 배포 작업은 늘 없다고 판단함
  • 배포 작업이 관리자 권한(sudo)으로 인증서 폴더를 확인
  • 결과: 인증서가 있으면 발급 도구를 부르지 않음
    • 계정 이관 때 옮겨 둔 인증서를 배포 작업이 바르게 알아봄
  • 머지 직후 dev 자동 배포
    • 서비스 동작은 같음. 배포 로그의 인증서 단계만 바뀜

Situation

인증서 확인이 늘 "없음"

확인 결과
/etc/letsencrypt/live·archive 권한 700 root
일반 사용자 [ -d ] 없음
sudo test -d 있음

Task

인증서 존재 확인을 실제 상태와 맞춤

Action

확인을 sudo test -d 로 교체

  • 배포 사용자는 비밀번호 없는 sudo 를 이미 씀 (같은 단계의 인증서 발급·파일 복사)

Result

확인 범위

  • 바뀐 원격 스크립트를 로컬에서 가짜 sudo 로 실행
    • 이관 모드에 인증서가 없으면 안내 후 exit 1
  • actionlint 새 지적 0건

후속

  • 머지 뒤 dev 배포 로그에서 "cert 이미 존재 — skip" 확인 예정

연관 이슈

- /etc/letsencrypt/live 확인을 sudo test -d 로 교체
  - 원인: live 는 700 root 라 ubuntu 의 [ -d ] 는 cert 가 있어도 거짓 (dev 박스 실측)
  - 결과(이전): 평소 배포는 매번 certbot 호출. 갱신 대상이 아니라 no action 으로 끝나 드러나지 않음
  - 결과(이전): #1219 의 이관 모드 분기는 cert 를 옮겨 둔 뒤에도 "cert 없음" 으로 배포를 멈춤
@m-a-king m-a-king added the fix 외부 가시적 결함 수정 label Oct 15, 2026
@m-a-king m-a-king self-assigned this Oct 15, 2026
@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: TeamPiKi/core/.coderabbit.yml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5ef9e958-23d6-49a7-972e-f190977179a9
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@m-a-king
m-a-king merged commit 50ddc1a into dev Oct 15, 2026
5 checks passed
@m-a-king
m-a-king deleted the fix/deploy-cert-check branch October 15, 2026 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 외부 가시적 결함 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant