Skip to content

Hot fix/ver1 2#50

Merged
choubung merged 4 commits intodevfrom
hot-fix/ver1-2
Nov 18, 2025
Merged

Hot fix/ver1 2#50
choubung merged 4 commits intodevfrom
hot-fix/ver1-2

Conversation

@choubung
Copy link
Copy Markdown
Owner

@choubung choubung commented Nov 18, 2025

Summary by CodeRabbit

릴리스 노트

  • Chores

    • Cloud SQL 데이터베이스 연결 구성을 업데이트했습니다.
  • Refactor

    • 보안 설정 구조를 간소화하고 통합했습니다.

choubung and others added 4 commits November 18, 2025 01:02
hot-fix: 자바 버전 충돌 crash 해결
- SecurityConfig에서 prod 프로필용 requiresSecure() 설정 제거
- Cloud Run의 TLS Termination으로 인한 HTTP/HTTPS 리디렉션 루프 방지
- HTTPS 처리는 Cloud Run 로드밸런서에 위임
@choubung choubung self-assigned this Nov 18, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Nov 18, 2025

Walkthrough

GitHub Actions 워크플로우의 Cloud SQL JDBC URL 설정을 변경하고, SecurityConfig에서 환경별 프로필 기반 SecurityFilterChain 빈을 제거하여 단일 통합 보안 필터 체인으로 리팩토링했습니다.

Changes

응집도 / 파일(들) 변경 요약
CI/CD 설정
.github/workflows/cd.yml
Cloud SQL JDBC URL을 호스트 google 사용에서 빈 호스트로 변경, socketFactory와 cloudSqlInstance에 의존하도록 수정
보안 설정 리팩토링
src/main/java/com/precourse/openMission/config/auth/SecurityConfig.java
환경별 @Profile 기반 prodFilterChain과 devFilterChain 빈 제거, 단일 public filterChain 메서드로 통합. requiresSecure() 강제 요구사항 및 commonSecurityConfig 헬퍼 메서드 제거

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • SecurityConfig.java 리팩토링: 프로필 기반 빈 분리 논리의 제거 및 단일 체인 통합 검증 필요
  • .github/workflows/cd.yml: Cloud SQL 연결 설정 변경이 배포 환경에서 정상 작동하는지 확인 필요

Possibly related PRs

Suggested labels

👩‍🔧 CI/CD, 🛠 Refactor

Poem

🐰 프로필을 벗겨내고 하나로 모으니,
보안 설정이 맑고 단순해졌네요!
클라우드 SQL도 새 길을 찾고,
복잡함 사라지고 우아함 남았어요. ✨
리팩토링 완성, 코드가 춤춘다!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check ❓ Inconclusive PR 제목이 변경사항의 핵심 내용을 명확하게 설명하지 못하고 있습니다. 'Hot fix/ver1 2'는 구체적이지 않으며, 실제 변경사항(Cloud SQL URL 수정, SecurityConfig 통합)을 반영하지 않습니다. 제목을 더 구체적으로 수정하세요. 예: 'Fix Cloud SQL connection and consolidate security configuration' 또는 변경사항을 명확히 설명하는 제목으로 변경해주세요.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hot-fix/ver1-2

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cefb390 and 1ff477d.

📒 Files selected for processing (2)
  • .github/workflows/cd.yml (1 hunks)
  • src/main/java/com/precourse/openMission/config/auth/SecurityConfig.java (2 hunks)
🔇 Additional comments (1)
.github/workflows/cd.yml (1)

79-79: Cloud SQL 연결 형식이 올바르게 업데이트되었습니다.

Cloud Run에서 Cloud SQL에 연결할 때 소켓 팩토리를 사용하는 것은 권장되는 방식입니다. 연결 URL 형식이 올바르게 설정되었습니다.


Comment @coderabbitai help to get the list of available commands and usage tips.

@choubung choubung added the 🚨 bug Something isn't working label Nov 18, 2025
@choubung choubung merged commit 72f08a8 into dev Nov 18, 2025
2 checks passed
@choubung choubung deleted the hot-fix/ver1-2 branch November 18, 2025 09:32
@choubung choubung added this to the 2️⃣ 배포 milestone Nov 19, 2025
@choubung choubung linked an issue Nov 19, 2025 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 bug Something isn't working 👩‍🔧 CI/CD

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

CI/CD: 배포

1 participant