Skip to content

feat: PROD 이미지 서버 분리#82

Merged
kon28289 merged 2 commits intodevfrom
feat/seperate-image-server
Mar 30, 2026
Merged

feat: PROD 이미지 서버 분리#82
kon28289 merged 2 commits intodevfrom
feat/seperate-image-server

Conversation

@kon28289
Copy link
Copy Markdown
Contributor

@kon28289 kon28289 commented Mar 30, 2026

🚀 1. 개요

  • PROD 서버의 이미지 서버를 DEV 서버와 분리합니다.
  • application-prod.yml의 ddl-auto를 validate로 변경합니다.

📝 2. 주요 변경 사항

📸 3. 스크린샷 (API 테스트 결과)

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 프로덕션 환경의 데이터베이스 스키마 검증 정책을 업데이트했습니다. 데이터베이스 테이블이 자동 생성되지 않고 기존 스키마 유효성만 검사하도록 변경됩니다.
    • 보안 관련 모듈 의존성을 최신 버전으로 업데이트했습니다.

@kon28289 kon28289 self-assigned this Mar 30, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 30, 2026

개요

프로덕션 환경의 Spring JPA Hibernate DDL 동작을 업데이트하여 spring.jpa.hibernate.ddl-autocreate에서 validate로 변경했습니다. 이는 데이터베이스 시작 시 스키마 자동 생성에서 기존 스키마 검증으로 변경합니다. 또한 보안 관련 서브모듈 커밋 참조를 업데이트했습니다.

변경 사항

Cohort / File(s) 요약
프로덕션 설정
src/main/resources/application-prod.yml
Spring JPA Hibernate DDL 설정을 create에서 validate로 변경하여 기존 스키마 검증으로 변경
보안 서브모듈
src/main/resources/security
Git 서브모듈 참조를 새로운 커밋으로 업데이트

예상 코드 리뷰 노력

🎯 1 (Trivial) | ⏱️ ~3 분

관련 가능 PR

추천 레이블

enhancement

추천 리뷰어

  • Juhye0k

시 🐰

프로덕션의 새로운 봄이 오네요 🌱
검증만 하는 현명한 선택
스키마를 소중히 지키고
보안은 더욱 견고하게 ✨
변화 속 안정을 찾다

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR 설명이 템플릿의 주요 섹션(개요, 주요 변경 사항)을 포함하고 있으나, '주요 변경 사항' 섹션이 비어있고 '스크린샷' 섹션도 미작성 상태입니다. '주요 변경 사항' 섹션에 변경된 항목들의 상세 설명을 추가하고, '스크린샷' 섹션에 API 테스트 결과나 검증 증거를 첨부해주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 핵심을 명확하게 반영하고 있으며, PROD 이미지 서버 분리라는 주요 변경 내용을 간결하게 요약하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/seperate-image-server

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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/main/resources/application-prod.yml`:
- Line 33: The production profile currently sets ddl-auto: validate which will
cause startup failures if schema and entities diverge; update project docs by
adding a "Production DDL change and deployment procedure" entry in README.md (or
a dedicated deploy guide) that references the production setting (ddl-auto:
validate) and details how to produce, version, review, test, and apply manual
DDL changes for entity updates (e.g., changes coming from User.java and
BaseEntity.java), including where SQL scripts are stored, how they are
version-controlled, pre-deployment validation steps, rollback steps, and
owner/approval flow; additionally, optionally propose adopting Flyway (or
Liquibase) as an automated alternative and note the absence of Flyway/Liquibase
dependencies and migration files in build.gradle.kts as a rationale.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 15052784-1ed8-4908-ae00-89cdb804d0ce

📥 Commits

Reviewing files that changed from the base of the PR and between 6fb6be4 and 6377ca5.

📒 Files selected for processing (2)
  • src/main/resources/application-prod.yml
  • src/main/resources/security

@kon28289 kon28289 requested a review from Juhye0k March 30, 2026 09:02
Copy link
Copy Markdown
Contributor

@Juhye0k Juhye0k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kon28289 kon28289 merged commit 9f8da30 into dev Mar 30, 2026
4 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.

2 participants