소셜 로그인 리팩토링

reading-log / backend

리딩로그 백엔드 레포지토리

3 stars 0 forks source link

Closed don9m1n closed 5 months ago

don9m1n commented 5 months ago

enjoy89 commented 5 months ago

사용자의 이메일과 소셜 로그인 플랫폼 정보를 조합하여 사용자를 구분하는 방법이 좋을거같은데, 어떻게 생각하시나요?

don9m1n commented 5 months ago

사용자의 이메일과 소셜 로그인 플랫폼 정보를 조합하여 사용자를 구분하는 방법이 좋을거같은데, 어떻게 생각하시나요?

그것도 좋을 것 같고, role 컬럼에 KAKAO, NAVER, GOOGLE을 추가해서 구분해도 좋을 것 같습니다. findByEmailAndRole() 메서드로 구분이 되지 않을까 싶네용

enjoy89 commented 5 months ago

넵 말씀해주신 부분처럼 새로 login_platform 이름으로 KAKAO, NAVER, GOOGLE 값들을 enum으로 저장해두고, findByEmailAndLoginPlatform 메서드로 유저를 찾도록 해보겠습니다!

enjoy89 commented 5 months ago

+스프링 시큐리티는 이메일 로그인 시 역할(권한)을 설정하는 로직에서 "ROLE_" 접두사가 붙어, DB에서 email+memberRole을 이용하여 사용자를 조회할 때 제대로 동작하지 않는 버그 발견

don9m1n commented 5 months ago

+스프링 시큐리티는 이메일 로그인 시 역할(권한)을 설정하는 로직에서 "ROLE_" 접두사가 붙어, DB에서 email+memberRole을 이용하여 사용자를 조회할 때 제대로 동작하지 않는 버그 발견

위키에 써놨습니다 ㅋㅋㅋㅋ