Skip to content

fix: SMTP 설정 누락 시 서버 기동 차단 - #108

Merged
hywznn merged 2 commits into
mainfrom
fix/55-smtp-config-validation
Aug 7, 2026
Merged

fix: SMTP 설정 누락 시 서버 기동 차단#108
hywznn merged 2 commits into
mainfrom
fix/55-smtp-config-validation

Conversation

@hywznn

@hywznn hywznn commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

왜 필요한가요?

PR #106의 후속 수정입니다. SMTP Provider를 선택했는데 연결 설정이 비어 있으면 서버는 정상 기동하고, 비밀번호 재설정 요청도 202를 반환하지만 실제 메일은 발송되지 않았습니다. 데모 시작 전에 설정 오류를 바로 발견할 수 있도록 기동 단계에서 차단합니다.

Refs #55
Refs #106

무엇이 바뀌나요?

  • SMTP Provider 활성화 시 Spring Mail host를 필수로 검증합니다.
  • SMTP 인증을 사용하는 경우 username과 password도 필수로 검증합니다.
  • 정상 SMTP 설정에서 Adapter가 선택되는 Spring Context 테스트를 유지합니다.
  • 빈 host와 인증정보 누락을 거부하는 테스트를 추가합니다.
  • DB와 Flyway는 변경하지 않습니다.

검증

  • SMTP 관련 테스트 통과
  • 전체 Gradle 테스트 통과
  • 빈 SMTP host 설정 시 Adapter 생성 및 서버 기동 차단 확인
  • GitHub Actions CI 통과

병합 전 확인

  • 리뷰 승인 후 Draft를 해제합니다.
  • 이 PR에서는 병합하지 않습니다.

SMTP Provider 활성화 시 host를 필수로 확인하고 인증 사용 시 username과 password 누락을 거부합니다. 정상 설정과 빈 host·인증정보 누락 테스트를 추가했습니다.
@hywznn
hywznn requested review from chaeliki and krestar August 7, 2026 05:57
@hywznn
hywznn marked this pull request as ready for review August 7, 2026 06:10

@krestar krestar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SMTP 활성화 시 필수 설정값을 기동 단계에서 검증하도록 잘 보완된 것 같습니다.
host 및 인증정보 누락 케이스 테스트도 확인했습니다

@hywznn
hywznn merged commit dac150a into main Aug 7, 2026
4 of 5 checks passed
@hywznn
hywznn deleted the fix/55-smtp-config-validation branch August 7, 2026 07:13
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