Skip to content

[fix/MAT-885] 비밀번호 찾기 JSON 파싱 에러 발생#360

Merged
sterdsterd merged 3 commits into
developfrom
fix/mat-885-find-password
May 27, 2026
Merged

[fix/MAT-885] 비밀번호 찾기 JSON 파싱 에러 발생#360
sterdsterd merged 3 commits into
developfrom
fix/mat-885-find-password

Conversation

@sterdsterd
Copy link
Copy Markdown
Collaborator

@sterdsterd sterdsterd commented May 27, 2026

Summary

비밀번호 찾기 플로우에서 빈 응답 body를 JSON으로 파싱하며 발생하던 JSON Parse error: Unexpected end of input 문제를 수정했습니다.
인증 코드 검증 응답의 value를 명시적으로 확인해 잘못된 코드로 다음 단계에 진입하지 않도록 막고, 코드/비밀번호 입력 CTA 활성화 조건을 실제 검증 조건과 맞췄습니다.
백엔드 최신 develop 기준으로 verify-code는 코드를 소비하지 않고 검증만 하며, 앱의 verify-code -> reset 구조와 호환됨을 확인했습니다.

Linear

Changes

  • 비밀번호 재설정 코드 발송/재설정 API의 빈 응답을 JSON으로 파싱하지 않도록 parseAs: 'stream'을 적용했습니다.
  • 인증 코드 검증 화면에서 data.value가 true일 때만 새 비밀번호 설정 화면으로 이동하도록 처리하고 실패 로그를 추가했습니다.
  • 인증 코드 입력을 숫자 키패드/6자리 제한으로 고정하고, 6자리 입력 완료 시에만 CTA가 활성화되도록 수정했습니다.
  • 새 비밀번호 입력은 8자 이상일 때만 CTA가 활성화되도록 수정하고 reset 실패 로그를 추가했습니다.

Testing

  • pnpm typecheck
  • 백엔드 /api/student/auth/password/reset/verify-code가 최신 develop에서 코드를 삭제하지 않고 BooleanResp(value)만 반환하는 구조임을 확인했습니다.
  • 실제 이메일 수신 및 운영 Redis TTL 만료 플로우는 로컬에서 수동 검증하지 못했습니다.

Risk / Impact

  • 영향 범위: 학생 앱 비밀번호 찾기 플로우의 인증 코드 발송, 코드 검증, 새 비밀번호 설정 화면
  • 확인이 필요한 부분: 틀린 인증 코드 입력 시 화면 전환이 막히는지, 올바른 코드로 검증 후 reset이 성공하는지, 만료/재발급 코드에 대한 에러 메시지가 기대 수준인지
  • 배포 시 유의사항: 백엔드의 verify-code가 코드를 소비하지 않는 최신 구현과 함께 배포되어야 verify-code -> reset 2-step 구조가 안정적으로 동작합니다.

Screenshots / Video

  • UI 구조 변경 없음

@linear
Copy link
Copy Markdown

linear Bot commented May 27, 2026

MAT-885

@sterdsterd sterdsterd self-assigned this May 27, 2026
@sterdsterd sterdsterd added the 🐞 Fix 버그 수정 label May 27, 2026
@claude
Copy link
Copy Markdown

claude Bot commented May 27, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@sterdsterd sterdsterd merged commit a44d4a2 into develop May 27, 2026
2 checks passed
@sterdsterd sterdsterd deleted the fix/mat-885-find-password branch May 27, 2026 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 Fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant