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

Refactor: 로그인 로직에 리팩토링, 수정 #102

Merged
merged 6 commits into from
Oct 18, 2023

Conversation

soulchicken
Copy link
Member

Refactor: 로그인 로직에 리팩토링, 수정

PR을 한 이유 🎯

  • �NEXT_PUBLIC 키워드로 시작하는 env는 브라우저에서도 확인할 수 있게 하는 키워드 컨벤션입니다.
  • 운영에 필요한 구글, 카카오, 네이버 측의 비즈니스 계정, 도메인 사용 허가같은 절차가 필요하고 해결될 때까지는 로그인을 막아야합니다.
  • Refresh 토큰이 적용되어야합니다.

변경사항 🛠

  • 소셜로그인에 사용한 key, client_id 환경변수에 NEXT_PUBLIC 키워드를 삭제했습니다.
  • 소셜로그인 운영에 필요한 절차가 완료될 때까지는 로그인을 막기 위해서 Toast를 띄웠습니다.
  • Refresh 토큰이 사용될 수 있도록 로직을 수정하고, 그 밖에 필요없는 Next-auth 코드를 삭제했습니다.

@soulchicken soulchicken merged commit f8afd1c into TeamATM:develop Oct 18, 2023
3 checks passed
@soulchicken soulchicken deleted the feature/login-refactoring branch October 18, 2023 06:39
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.

1 participant