Feature/oauth-setting(wr9-24)

[leeys9423]

Pull Request Template

📝 PR 설명

• OAuth2.0 관련 기본 클래스 파일 구현과 설정 적용 (env 파일 최신화 하셔야 합니다)

🔍 주요 변경사항

• OAuth 관련 공통 에러추가
• Member, SocialAccount 관련 엔티티 생성
• application.yml 파일 수정 ( OAuth 추가 설정 + api key값 추가 )
• OAuth 추상 클래스 생성 후 제공하는 소셜 구체 클래스 구현

📸 스크린샷 (선택사항)

• UI 변경사항이 있다면 스크린샷을 첨부해주세요.

🧪 테스트

• 단위 테스트 추가/수정
• 테스트 커버리지 확인
• 소나클라우드 품질 게이트 통과

✅ 체크리스트

• 코드 컨벤션을 준수했나요?
• 불필요한 코드나 주석이 없나요?
• 브랜치 네이밍 컨벤션을 따랐나요?
• 관련 문서를 업데이트했나요?

🤝 관련 이슈

• closes [WR9-24] OAuth2 기본 설정 구현 #23

[sonarqubecloud]

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Kojinyoung7220]

column(nullable = false) 대신 NotNull을 사용하는 거에 대해선 어떻게 생각하시나요?

[leeys9423]

NotNull은 validation에 관련된 어노테이션이라고 알고있습니다!

해당 설정은 엔티티 잖아여~ 그래서 컬럼에 대한 정의를 내려주고 있는거에요!
이렇게 설정해야만 Table이 생성될 때 Null 미허용으로 적용되는 걸로 알고있습니다

[Kojinyoung7220]

아 그렇게 상관은 없는 거 같습니다
https://hereishyun.tistory.com/166
이런 의견이 있어 여쭤봤습니당

[Kojinyoung7220]

Member가 생성이 안될 경우가 있을까요? (ex 비밀번호에 null이 들어갈 경우)
만약 있으면 생성이 실패해서 생기는 경우도 테스트 코드로 있으면 좋을 거 같습니다~

[leeys9423]

일단 비즈니스 로직적으로 생각해봐야 할 부분이 있을 것 같아요!

현재 우리 서비스는 소셜로그인만 제공하고 있죠!

따라서 password라는 필드는 사실상 사용되고 있질 않습니다!

하지만 password라는 필드를 따로 빼놓은 이유는 관리자는 그럼 어떻게 로그인해야하지? 관리자는 소셜로그인을 사용하지 않을까? 라는 생각에 필드를 생성해 놓은 거였어요. 금일 회의로 일단 관리자 쪽은 따로 생성하지 않는 방향으로 얘기가 됐으므로 근본적인 password 필드는 삭제가 될 예정입니다!

[Kojinyoung7220]

알겠습니다~!