Skip to content

[Gibeom] Week4 미션#19

Open
kite-pp wants to merge 4 commits intoGibeomfrom
Gibeom-Week4
Open

[Gibeom] Week4 미션#19
kite-pp wants to merge 4 commits intoGibeomfrom
Gibeom-Week4

Conversation

@kite-pp
Copy link
Copy Markdown

@kite-pp kite-pp commented Apr 6, 2026

🔗 연관 이슈

🛠 작업 내용

🖼 스크린샷 (선택)

👀 리뷰 요구사항 (선택)

🤖 AI 활용

  • AI 사용 안 함
  • 코드 작성 아이디어 참고
  • 테스트/리팩토링 보조
  • 문서/주석 작성 보조
  • 기타 (아래에 간단히 작성)

💬 나의 프롬프트

패키지 구성 도와줘

🧠 AI 응답

도메인 기반 아키텍쳐의 패키지 설계를 도와줌.

✅ 내가 최종 선택한 방법 (이유)

💡 나만의 Tip (선택)

@kite-pp kite-pp self-assigned this Apr 6, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1057d854dc

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

driver-class-name: com.mysql.cj.jdbc.Driver # MySQL JDBC ???? ??? ??
url: ${DB_URL:jdbc:mysql://localhost:3306/umc10th?serverTimezone=Asia/Seoul&allowPublicKeyRetrieval=true&useSSL=false} # jdbc:mysql://localhost:3306/{???????}
username: ${DB_USER:root} # MySQL ?? ??
password: ${DB_PW:patrashu370!} # MySQL ????
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge application.yml 기본 비밀번호 하드코딩을 제거하세요

application.yml${DB_PW:...} 기본값에 실제 비밀번호를 넣어두면 환경변수가 비어 있을 때도 민감정보가 코드로 배포되어 노출됩니다. Spring 설정에서 기본값은 안전한 개발용 값 또는 미설정 상태로 두고, 비밀번호 누락 시 애플리케이션이 명확히 실패하도록 만드는 것이 올바른 습관입니다. 기본값을 제거해 ${DB_PW}로 변경하고, 프로필별 설정/검증으로 필수값 누락을 잡아주세요. 다음으로 프로필 기반 설정 분리와 구성값 검증을 학습해보세요.

Useful? React with 👍 / 👎.

@@ -0,0 +1,248 @@
#!/bin/sh
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge gradlew 실행 권한을 커밋에 반영하세요

이 커밋에서 gradlew가 실행 비트 없이 추가되어(100644) Linux/macOS에서 ./gradlew 실행 시 Permission denied가 발생해 빌드/테스트 시작 자체가 막힙니다. Spring 학습 프로젝트에서도 래퍼 스크립트는 누구나 동일하게 실행 가능해야 온보딩과 CI 흐름을 재현할 수 있습니다. chmod +x gradlew 후 실행 권한 변경까지 커밋해 주세요. 다음 학습 주제로는 Gradle Wrapper와 Git 파일 권한(모드 비트)를 추천합니다.

Useful? React with 👍 / 👎.

@@ -0,0 +1,4 @@
package PACKAGE_NAME;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

package 경로는 명시하는 것이 권장됩니다.

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.

3 participants