Skip to content

[Chore] 초기 세팅 오류 수정#4

Open
Sehi55 wants to merge 3 commits into
devfrom
fix/initial-setting
Open

[Chore] 초기 세팅 오류 수정#4
Sehi55 wants to merge 3 commits into
devfrom
fix/initial-setting

Conversation

@Sehi55
Copy link
Copy Markdown

@Sehi55 Sehi55 commented Apr 19, 2026

  • application.yaml, build.gradle 오류 수정

@Sehi55 Sehi55 self-assigned this Apr 19, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 19, 2026

📝 Walkthrough

Summary by CodeRabbit

릴리스 노트

  • Chores
    • JWT 라이브러리 버전을 0.12.3에서 0.12.6으로 업데이트했습니다.
    • 데이터베이스 배치 처리 설정 구조를 변경했습니다.
    • 운영 환경에서 필수 환경 변수 설정을 업데이트했습니다.

워크스루

JWT 라이브러리(jjwt-api, jjwt-jackson)를 0.12.3에서 0.12.6으로 업그레이드하고, Spring JPA/Hibernate 설정을 spring.jpa.properties.hibernate 네임스페이스로 정리했습니다. 프로덕션 환경에서 MEETING_SECRET_KEY 환경변수 설정을 필수화했습니다.

변경 사항

Cohort / File(s) 요약
의존성 업그레이드
mannabom-server/build.gradle
jjwt-api, jjwt-jackson을 0.12.3에서 0.12.6으로 업그레이드하여 jjwt-impl 버전과 정렬
Hibernate 설정 정리
mannabom-server/src/main/resources/application.yaml, application-prod.yml
default_batch_fetch_size, use_sql_comments 설정을 spring.jpa.properties.hibernate 구조로 통합 정리
프로덕션 환경 보안 강화
mannabom-server/src/main/resources/application-prod.yml
meeting.cursor.secret 기본값 제거하여 MEETING_SECRET_KEY 환경변수 필수화

예상 코드 리뷰 노력

🎯 2 (Simple) | ⏱️ ~10 minutes

🐰 버전을 올리고, 설정을 정리했네,
보안도 한 층 강화되어 좋구나!
프로퍼티들이 제 자리를 찾았고,
JWT도 새롭게 옷을 입었으니,
우리의 서버는 더욱 견고해졌네!

🚥 Pre-merge checks | ✅ 1 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive 제목이 변경 사항의 주요 내용과 부분적으로만 관련되어 있습니다. '초기 세팅 오류 수정'은 매우 일반적인 표현이며, JWT 라이브러리 버전 업데이트와 Hibernate 설정 경로 변경 같은 구체적인 변경 사항을 충분히 반영하지 못합니다. 제목을 더 구체적으로 수정하세요. 예: '[Chore] JWT 라이브러리 업데이트 및 Hibernate 설정 경로 통일' 같이 주요 변경 내용을 명확히 표시하는 것이 좋습니다.
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed 설명이 변경 사항과 관련이 있습니다. application.yaml과 build.gradle 수정을 언급하고 있으며, 이는 실제 변경 사항과 일치합니다.

✏️ 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 fix/initial-setting

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

Copy link
Copy Markdown

@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

🧹 Nitpick comments (1)
mannabom-server/src/main/resources/application-prod.yml (1)

97-97: 운영 비밀키 기본값 제거는 보안 측면에서 좋은 변경입니다.

Line 97 변경으로 운영 환경에서 의도치 않은 기본키 사용 위험이 줄었습니다. 배포 파이프라인/시크릿 매니저에 MEETING_SECRET_KEY 주입만 체크리스트로 고정해두면 더 안전합니다.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@mannabom-server/src/main/resources/application-prod.yml` at line 97, You
removed the default production secret (the property "secret:
${MEETING_SECRET_KEY}" in application-prod.yml) which is good; now ensure the
deployment and runtime validate that MEETING_SECRET_KEY is provided by (1)
adding a required-secret check in your deployment pipeline/secret manager
checklist and (2) adding a startup validation in the application bootstrap
(check the "secret" property read from MEETING_SECRET_KEY and fail-fast with a
clear error if missing) so the app never runs with a missing/empty
MEETING_SECRET_KEY.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@mannabom-server/build.gradle`:
- Around line 31-33: Update the JJWT dependency versions declared
(io.jsonwebtoken:jjwt-api, io.jsonwebtoken:jjwt-impl,
io.jsonwebtoken:jjwt-jackson) from 0.12.6 to a newer release (preferably 0.13.0
or at least 0.12.7) so all three coordinates match the chosen version; modify
the three implementation entries to the new version, refresh/reload Gradle
dependencies and run the build/tests to verify no API breaks.

---

Nitpick comments:
In `@mannabom-server/src/main/resources/application-prod.yml`:
- Line 97: You removed the default production secret (the property "secret:
${MEETING_SECRET_KEY}" in application-prod.yml) which is good; now ensure the
deployment and runtime validate that MEETING_SECRET_KEY is provided by (1)
adding a required-secret check in your deployment pipeline/secret manager
checklist and (2) adding a startup validation in the application bootstrap
(check the "secret" property read from MEETING_SECRET_KEY and fail-fast with a
clear error if missing) so the app never runs with a missing/empty
MEETING_SECRET_KEY.
🪄 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: 821586d8-8563-431d-accb-01a484794904

📥 Commits

Reviewing files that changed from the base of the PR and between 9ed14be and 44151a5.

📒 Files selected for processing (3)
  • mannabom-server/build.gradle
  • mannabom-server/src/main/resources/application-prod.yml
  • mannabom-server/src/main/resources/application.yaml

Comment thread mannabom-server/build.gradle
@kimjuneon kimjuneon added the bug Something isn't working label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants