Skip to content

feat: 사용자 Email, Oauth 로그인시 정지된 계정에 에러 표기#40

Open
vpvm96 wants to merge 2 commits into
developfrom
user-suspense
Open

feat: 사용자 Email, Oauth 로그인시 정지된 계정에 에러 표기#40
vpvm96 wants to merge 2 commits into
developfrom
user-suspense

Conversation

@vpvm96

@vpvm96 vpvm96 commented May 10, 2026

Copy link
Copy Markdown
Collaborator

무림에 고하노라

"강호에 바람이 분다. 이 PR이 병합되면, 세상이 달라지리라."

어떤 무공을 펼쳤는가 (변경 사항)

  • 사용자 Email, Oauth 로그인시 정지된 계정의 경우 에러 표기 작업

왜 이 초식을 택했는가 (변경 이유)

  • 관리자에 사용자 정지 / 활동 API 추가됨에 따라 연계 작업

내공 점검 (테스트)

  • [ v] 단위 시험을 통과하였는가
  • [ v] 빌드에 이상이 없는가
  • [ v] 로컬에서 직접 손으로 확인하였는가

비급 참고 (관련 문서/이슈)

문파에 전할 말 (리뷰어에게)

"검을 뽑기 전에 한 번 더 살펴주시오."

  • 이 PR 템플릿은 머고...?

@vpvm96 vpvm96 requested a review from FoxMon May 10, 2026 07:48
@github-actions

Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Health
Hammer.User.Api 46% 19%
Hammer.User.Application 84% 89%
Hammer.User.Domain 89% 72%
Hammer.User.Infrastructure 18% 35%
Summary 37% (936 / 2537) 39% (182 / 462)

if (user.Email is null)
throw new UnauthorizedException(InvalidCredentialsMessage);

if (user.Status == UserStatus.Suspended)

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.

user entity에 rich도메인으로

if (user.Status == UserStatus.Suspended)
throw new ForbiddenException(SuspendedMessage);

if (user.Status != UserStatus.Active)

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.

위와 마찬가지

var user = await userRepository.GetByIdAsync(userId, ct)
?? throw new UnauthorizedException("사용자를 찾을 수 없습니다.");

if (user.Status == UserStatus.Suspended)

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.

위와 동일

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