Skip to content

CICD: 배포 전 환경변수 추가#71

Merged
philbae0 merged 1 commit into
developfrom
chore/#70
Jul 26, 2026
Merged

CICD: 배포 전 환경변수 추가#71
philbae0 merged 1 commit into
developfrom
chore/#70

Conversation

@philbae0

Copy link
Copy Markdown
Collaborator

Summary

  • 배포 서버(EC2) 환경변수 업데이트 (Clova/S3 관련 추가)

Related Issue

Describe your code

  • 작업 내용 (What I Did) : GitHub Secrets 변수 신규 등록, deploy.yml 파일 환경변수 추가 반영
  • 스크린샷/결과 (Optional) : API 테스트 결과 첨부
  • 논의사항/질문 (To Reviewers) : AI 서버가 아직 배포 전이라 GitHub Secret에 AI_SERVER_BASE_URL은 등록하지 않았고, 기본값 로컬로 처리됩니다.

Checklist

  • 리뷰어 등록

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 5900ce2a-f86a-41bd-8307-eedcb454bc6c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/#70

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

🛡️ 문제점

  • SCP 명령어에서 -o StrictHostKeyChecking=no를 사용하면 SSH 호스트 키 검증이 비활성화되어, 배포가 중간자 공격(man-in-the-middle attacks)에 노출될 수 있습니다.

💡 개선 사항

  • 스크립트의 가독성과 정리 상태를 위해 환경 변수 선언을 서비스별(AWS, Clova, AI)로 논리적으로 그룹화하세요.
  • 워크플로우 스크립트에서 환경 변수 전달 시 단일 소스 환경 내보내기를 활용하여 반복을 줄이세요(DRY 원칙).
  • YAML 파일 전반에 걸쳐 속성 명명 규칙을 표준화하여 환경 간 일관성을 확보하세요.

@philbae0
philbae0 merged commit aa80202 into develop Jul 26, 2026
2 checks passed
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.

🔄[CI/CD] 배포 전 환경변수 추가

3 participants