Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEAT] 비밀번호 찾기 기능을 위한 사용자 인증 체계 구현 #94

Merged
merged 8 commits into from
Nov 20, 2024
Merged

[FEAT] 비밀번호 찾기 기능을 위한 사용자 인증 체계 구현 #94

merged 8 commits into from
Nov 20, 2024

Conversation

liimjiin
Copy link
Contributor

@liimjiin liimjiin commented Nov 19, 2024 •
edited
Loading

관련 이슈

변경 사항

  1. 사용자의 그룹을 조회하는 /check-user 구현

    • 비로그인 상태에서 사용자 권한 검증을 위한 엔드포인트
    • 이메일 기반으로 Cognito 사용자 그룹 정보 조회

  2. 에러 처리 체계 개선

    • 사용자 상태 관련 (AUTH-005 ~ 007)
    • 인증 코드 관련 (AUTH-008 ~ 010)
    • 비밀번호 관련 (AUTH-011 ~ 016)

체크리스트

  • 코드 컨벤션을 준수하였습니까?
  • 모든 테스트를 통과하였습니까?
  • 관련 문서를 업데이트하였습니까?

공유사항

  • 빠른 시일 내에 비로그인/로그인의 비밀번호 변경 기능의 차이점에 대한 문서를 공유할 수 있도록 하겠습니다.

추후 업무

  • 마이페이지

@liimjiin
FEAT: 사용자 확인하는 엔드포인트 접근 허용
740783d
@liimjiin
REFACTOR: 에러 코드 정의 구조 개선 및 카테고리 세분화
4d3c3cf 
- 인증 관련 에러 코드를 사용자 상태, 인증 코드, 비밀번호로 세분화
- 에러 코드 번호 순서 정리 및 일관성 유지
- 각 카테고리별 그룹화로 가독성과 유지보수성 향상
@liimjiin
FEAT: Cognito 사용자 그룹 조회 기능 추가
6fff6b1 
- 이메일을 통해 Cognito 사용자 정보를 검색하여 그룹 목록 반환
- AdminListGroupsForUserRequest를 사용하여 사용자 그룹 조회
- UserNotFoundException에 대한 사용자 정의 예외 처리
- 예외 발생 시 INTERNAL_SERVER_ERROR 반환
@liimjiin
FEAT: 사용자 그룹 조회를 위한 /check-user 엔드포인트 추가
9a6197d 
- 이메일을 입력받아 해당 사용자의 Cognito 그룹 정보를 반환하는 기능 추가
- AuthService의 `getUserGroups` 메서드를 활용하여 그룹 조회 구현
- 기존 컨트롤러 구조 유지 및 응답 형식 통일
@liimjiin
Merge branch 'main' of https://github.com/SeSAC-Ermes/Epari-backend i…
92b642e 
…nto feature/47-user-password-update-and-find
@liimjiin
Merge branch 'main' of https://github.com/SeSAC-Ermes/Epari-backend i…
bed70be 
…nto feature/47-user-password-update-and-find
@liimjiin
Merge branch 'main' of https://github.com/SeSAC-Ermes/Epari-backend i…
1a687fb 
…nto feature/47-user-password-update-and-find
@liimjiin
Merge branch 'main' of https://github.com/SeSAC-Ermes/Epari-backend i…
4e2321f 
…nto feature/47-user-password-update-and-find
@liimjiin liimjiin added the enhancement New feature or request label Nov 19, 2024
@liimjiin liimjiin self-assigned this Nov 19, 2024
12ka39
12ka39 approved these changes Nov 20, 2024
View reviewed changes
Copy link
Contributor

@12ka39 12ka39 left a comment

Choose a reason for hiding this comment

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

비밀번호 찾기 기능을 위해 사용자 인증 체계 구현하시고
에러 처리도 상세하게 나누셨군요. 고생하셨습니다!

cuteJJong
cuteJJong approved these changes Nov 20, 2024
View reviewed changes
Copy link
Contributor

@cuteJJong cuteJJong left a comment

Choose a reason for hiding this comment

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

보안 계층을 잘 구현하시고 예외 처리도 꼼꼼하게 잘 해주셨고 너무 고생하셨습니다.

@liimjiin liimjiin merged commit 485dd88 into main Nov 20, 2024
1 check passed
@liimjiin liimjiin deleted the feature/47-user-password-update-and-find branch November 20, 2024 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cuteJJong cuteJJong cuteJJong approved these changes

@12ka39 12ka39 12ka39 approved these changes

@Chan-GN Chan-GN Awaiting requested review from Chan-GN

Assignees

@liimjiin liimjiin

Labels
enhancement New feature or request
Projects
Epari-backend
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 비밀번호 찾기 기능을 위한 사용자 인증 체계 구현
3 participants
@liimjiin @cuteJJong @12ka39