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

Feat/#147/ 비밀번호 암호화 #148

Merged
merged 3 commits into from
Sep 11, 2024
Merged

Conversation

drbug2000
Copy link
Collaborator

📝 요약

사용자의 비밀번호를 암호화
회원가입 로그인에 적용
BCryptPasswordEncoder 사용

🔖 변경 사항

  • SecurityConfig 수정
  • endcoding 과정 회원가입에 적용
  • 로그인에도 적용
  • validatePassword 함수 수정
  • User.passMatch 함수 삭제

✅ 리뷰 요구사항

해당 코드를 수정하게 되면 DB를 초기화 해야하기 때문에 유심히 봐주세요.
이 PR은 추후 RDB 관련 cd 과정을 수정하면 merge 하겠습니다.



📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@drbug2000 drbug2000 linked an issue Sep 7, 2024 that may be closed by this pull request
3 tasks
seongjunnoh
seongjunnoh approved these changes Sep 8, 2024
@hyunn522 hyunn522 merged commit c264c7f into develop Sep 11, 2024
3 checks passed
@hyunn522 hyunn522 deleted the feat/#147/passwordencode branch September 30, 2024 00:18
seongjunnoh pushed a commit that referenced this pull request Oct 30, 2024
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.

Feat : 비밀번호 암호화 passwordEncoder적용
3 participants