feat: 비밀번호 재설정과 가입 약관 동의 이력 구현 - #100
Merged
Merged
Conversation
비밀번호 재설정 토큰과 약관 동의 이력을 위한 V24 스키마를 추가합니다. PostgreSQL tenant 정책과 안전한 토큰 생성·해시·알림 Port를 구성하고 관련 설정값을 등록합니다.
회원가입 필수 약관과 버전을 검증해 이력을 기록합니다. 비밀번호 재설정 요청·완료 API에 계정 노출 방지, 일회용 토큰, 동시 요청 제어, IP 제한, 비밀번호 변경 및 기존 세션 폐기를 연결합니다.
약관 동의, 존재 여부를 숨긴 재설정 요청, 토큰 만료·재사용·동시 요청, 기존 세션 폐기와 OpenAPI 계약을 검증합니다. PostgreSQL migration과 RLS 격리 검증에도 신규 테이블과 bootstrap 함수를 포함합니다.
비밀번호 재설정 만료·재요청·요청 제한 값과 가입 약관 버전 환경 변수를 예시 파일에 문서화합니다. 실제 알림 Provider가 후속 연동 대상임을 명시합니다.
hywznn
marked this pull request as ready for review
August 6, 2026 10:32
비밀번호 변경 성공 시 같은 사용자의 모든 미사용 재설정 토큰을 한 번에 무효화합니다.\n계정 잠금을 먼저 획득해 서로 다른 링크의 동시 사용도 한 요청만 성공하도록 정리합니다.\n알림 발송을 전용 비동기 실행기로 분리하고 Provider 실패가 202 응답에 영향을 주지 않도록 격리합니다.\n이전 링크 거부와 알림 실패 시 동일 응답을 검증하는 통합 테스트를 추가합니다.
krestar
approved these changes
Aug 6, 2026
krestar
left a comment
Contributor
There was a problem hiding this comment.
비밀번호 재설정의 보안·동시성 처리와 약관 동의 이력, 마이그레이션 구성을 확인했습니다.
현재 범위에서 blocking issue는 없어 승인합니다.
chaeliki
approved these changes
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
왜 필요한가요?
Figma의 비밀번호 찾기·회원가입 화면이 실제 Server API와 안전하게 연결되도록 인증 복구와 약관 동의 이력을 구현합니다.
Refs #55
무엇이 바뀌나요?
POST /api/v1/auth/password-reset-requests를 추가했습니다.POST /api/v1/auth/password-resets를 추가했습니다.보안·호환성
검증
./gradlew clean test성공병합 순서
이 PR의 V24·V25를 먼저 병합한 뒤 #54의 V26·V27, #85의 V28 순서로 병합해야 합니다.