Skip to content

Comments

feat: 이메일 인증 기능 구현#63

Open
hodoon wants to merge 14 commits intodevfrom
feat/ALT-142
Open

feat: 이메일 인증 기능 구현#63
hodoon wants to merge 14 commits intodevfrom
feat/ALT-142

Conversation

@hodoon
Copy link
Contributor

@hodoon hodoon commented Feb 8, 2026

@hodoon hodoon requested review from juny0955 and ysw789 February 8, 2026 22:18
@hodoon hodoon self-assigned this Feb 8, 2026
@hodoon hodoon added the FEAT label Feb 8, 2026
Copy link
Contributor

@ysw789 ysw789 left a comment

Choose a reason for hiding this comment

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

  • 이메일 인증 Rate Limit 구현은 아주 잘 하셨습니다~
  • 지금은 이메일 발송할 때 성공 / 작업 진행중인 내역도 log.info()로 로그에 남게 하고 있는데, 실패한 것만 로그에 남기면 어떨까 합니다. 성공한 것도 로깅하면 너무 과도하게 로그가 찍힐 것 같습니다.
  • 클래스 네이밍에서 Port는 저희 프로젝트에서 사용하지 않는 네이밍입니다. 기존 outbound 포트 관례에 맞게 아래와 같이 수정해 주세요.
    기존 패턴
    domain/notification/port/outbound/NotificationRepository.java         ← 영속성
    domain/notification/port/outbound/NotificationQueryRepository.java    ← 쿼리
    domain/auth/port/outbound/KakaoAuthClient.java                        ← 외부 API
    
    변경 요청
    EmailSendLogPort                → EmailSendLogRepository
    EmailVerificationTokenStorePort → EmailVerificationTokenStoreRepository
    EmailSenderPort                 → EmailClient (KakaoAuthClient, AppleAuthClient 패턴 참고)
    
  • 추가된 env 값은 노션 문서에 최신화 해주시고, DB 마이그레이션 DDL은 작업 문서에 남겨주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants