-
Notifications
You must be signed in to change notification settings - Fork 4
#260-3 [Feat] 로그인 / 회원가입 기능 구현 #263
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
#260-3 [Feat] 로그인 / 회원가입 기능 구현 #263
Conversation
- 비밀번호 변경 - 유저 메타 데이터 업데이트 기능 - 로컬로 로그인할 수 있도록 하는 회원가입 기능
- 아이디로 엔티티 검색 가능. - 추후 인덱싱해야함
- 1초는 1000 밀리 세컨드 ..
- 현재 시간을 더하는게 아니라, 만료 전체 시간을 더해야했음 - jwt service 설정에 만료시간 설정이 없어서 삭제
- `인증`까지만 범위 설정을 어떻게 할지 고민을 많이 했음 - 인증에 성공했다면, 토큰을 받아오도록 통일해야하지 않을까? 라는 생각을 하게 됨 - 그렇게 전략 패턴별로 토큰을 반환하도록 통일함 - GithubProfile 이라는 정보도 나중에 삭제할 예정
- 회원가입을 위한 DTO 추가 - 로그인을 위한 DTO 추가
- 로컬 로그인 시 http 에서도 테스트할 수 있도록 허용
yiseungyun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
매직 남바를 사용하다니 고수의 냄새가 납니다.
MZTM 드립니다. 다봉. 👍
twalla26
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!!
ShipFriend0516
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
로컬 로그인 막아뒀었는데 봉인 해제할 때가 되었군요.. 고생하셨습니다
Note
한꺼번에 많은 PR을 올리지 않도록 분리하였습니다. (제가 코드를 빨리치다보니 이걸 좀 분리하고싶은데 고민이네요.)
#260
#261
을 먼저 확인해주세요. 이 PR과 연관되어있어서 이 PR 이 머지되면 저곳도 머지 됩니다.
관련 이슈 번호
✅ 체크리스트
🧩 작업 내용
📝 작업 상세 내역
로그인 회원가입 기능을 위한 DTO 추가
저한테 말 거시면 상세히 알려드리겠습니다.로그인 기능을 위한 passport 전략 추가
auth에서 합니다.쿠키 설정 시 secure 값을 production 일때만 되도록 허용
NODE_ENV를 설정하게 하는건 [Chore] passport-local 의존성 추가, NODE_ENV 환경 제공 #259 여기서 추가했습니다.📌 테스트 및 검증 결과
💬 다음 작업 또는 논의 사항
🐥 리뷰 받고 싶은 포인트