Skip to content

fix(infra): 운영 배포 Secret 전달 및 CI 안정성 개선 - #208

Merged
1Seob merged 2 commits into
developfrom
fix/infra-prod-deploy-secrets
Jul 28, 2026
Merged

fix(infra): 운영 배포 Secret 전달 및 CI 안정성 개선#208
1Seob merged 2 commits into
developfrom
fix/infra-prod-deploy-secrets

Conversation

@1Seob

@1Seob 1Seob commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

📝 요약(Summary)

  • 운영 EC2 배포에서 SSH 액션 실행 전 발생한 Argument list too long 오류를 수정했습니다.
  • Secret 값이 하나의 INPUT_SCRIPT에 합쳐지지 않도록 환경변수 전달 구조를 개선했습니다.
  • UTC 기반 CI에서 특정 시간대에 실패하던 일일 리포트 테스트를 안정화했습니다.

주요 변경사항

  • 운영 배포
    • Base64 프롬프트와 애플리케이션 설정 50개를 envenvs로 개별 전달
    • SSH 스크립트 내부의 ${{ secrets.* }} 직접 치환 제거
    • INPUT_SCRIPT가 Secret 크기에 비례해 증가하던 구조 개선
  • 배포 보안
    • set -x를 제거해 디코딩된 프롬프트와 민감 정보의 로그 노출 위험 완화
    • 기존 프롬프트 디코딩과 Java 환경변수 상속 동작 유지
  • CI 테스트 안정성
    • DailyReportServiceTest의 날짜 기준을 TodayDateTimeProvider로 통일
    • UTC 러너와 Asia/Seoul 비즈니스 날짜 차이로 발생하던 Mockito stub 인자 불일치 해결

검증

  • 운영 배포 워크플로 YAML 파싱 성공
  • envenvs 전달 변수 50개 일치 및 누락·중복 0건 확인
  • SSH 스크립트 내부 Secret 직접 치환 0건 확인
  • DailyReportServiceTest 집중 테스트 통과
  • 전체 Gradle 테스트 256개 통과
  • 실패 및 오류 0건

🔗 Related Issue

  • Closes:

💬 공유사항

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • PR 제목을 커밋 메시지 컨벤션에 맞게 작성했습니다.

1Seob added 2 commits July 29, 2026 02:52
SSH 스크립트에 직접 치환하던 Secret을 env와 envs를 통한 개별 환경변수 전달 방식으로 변경하고, INPUT_SCRIPT 크기 초과로 발생한 운영 배포 실패 해결 및
셸 명령 추적 비활성화를 통한 민감 정보 노출 방지
DailyReportServiceTest의 날짜 기준을 TodayDateTimeProvider로 통일해 UTC 기반 CI에서 발생하던 Mockito stub 인자 불일치 해결
@1Seob 1Seob changed the title fix(infra): 운영 배포 Secret 전달 방식 개선 fix(infra): 운영 배포 Secret 전달 및 CI 안정성 개선 Jul 28, 2026
@1Seob
1Seob merged commit 7d142b9 into develop Jul 28, 2026
1 check passed
@1Seob
1Seob deleted the fix/infra-prod-deploy-secrets branch July 28, 2026 18:17
@1Seob 1Seob mentioned this pull request Jul 28, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant