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.

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

Password validate 추가 #12

Merged
merged 6 commits into from
May 7, 2024
Merged

Password validate 추가 #12

merged 6 commits into from
May 7, 2024

Conversation

HwiYul-G
Copy link
Owner

@HwiYul-G HwiYul-G commented May 7, 2024

  1. 문제
    • 등록할 때와 로그인할 때 unsafe password warning이 뜨는 문제가 있었다.
  2. 해결 방안 계획
    • 서버에서 프론트로 비밀번호를 보내는 문제가 있다는 이야기를 들어서 코드를 찾아봤다.
    • 코드상에 문제가 없다는 것을 확인하고 나서, 테스트할 때 사용한 비밀번호가 1234여서 추측하기 쉬운 비밀번호여서 그랬다는 것을 알게 되었다.
    • 비밀번호 규칙을 5글자 이상 이 중 한 글자는 영어를 포함하도록 정했다.
  3. 해결 방안 계획 중 생긴 문제와 해결
  • 기존에 UserDto에서 이메일 검증을 하고 있었기 때문에 이 곳으로 비밀번호 검증을 배치하고 Exception을 throw 시켰다.
  • UserController에 userDto가 파라미터로 들어오기 때문에 함수 내부에 있는 try-catch에서 잡히지 않는 것을 발견했다.
  • UserValidator를 별도로 만들어서 service 내에서 검증을 하도록 만들었다.
  • catch에서 잡아서 오류 메시지를 redirectAttribute로 보내고, view에서 이것을 사용자에게 보여주게 했다.

@HwiYul-G HwiYul-G merged commit 1bc7dd4 into main May 7, 2024
1 check passed
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.

1 participant