PR: #29 · 리뷰어: 준용 + 유나 (둘 다 CHANGES_REQUESTED) · 해결:
f410173
개요: 인증 시스템 전면 구현 PR에서 10건 이슈가 발견됨. 보안 관련 이슈가 다수 포함되어 두 리뷰어 모두 CHANGES_REQUESTED.
1. REST_FRAMEWORK 중복 선언
settings.py에 REST_FRAMEWORK dict 2번 선언 → 인증 설정 덮어씀.
→ 단일 dict로 병합
2. 더미 URL import 누락 → NameError
urls.py에서 import 없이 view 참조 → 서버 시작 시 NameError.
→ import 추가
3. django-axes requirements.txt 누락 → ModuleNotFoundError
settings.py에 django-axes 설정은 있으나 requirements.txt에 미등록.
→ requirements.txt에 추가
4. Google OAuth 400 에러
Google OAuth 콜백 URL 설정 오류 → 인증 플로우 실패.
→ 콜백 URL 수정
5. FindUsernameView .first() → 동일 이메일 다중 계정 미처리
같은 이메일로 여러 계정 존재 시 .first()로 첫 번째만 반환.
→ 다중 계정 처리 로직 추가
6. GoogleLoginButton 환경변수 미설정 크래시