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

Feature/#98 SNS 로그인 #82

Merged
merged 7 commits into from
Jul 20, 2023
Merged

Feature/#98 SNS 로그인 #82

merged 7 commits into from
Jul 20, 2023

Conversation

chaeseungyun
Copy link
Contributor

@chaeseungyun chaeseungyun commented Jul 11, 2023

SNS 로그인을 구현했습니다.

각 SNS 로그인 버튼을 누르면 인가 코드와 함께 리다이렉트 되는데 이때 받은 인가 코드로 엑세스 토큰을 발급 받습니다.
따라서 각 리다이렉트 URL에 맞는 컴포넌트를 만들고 라우팅 했습니다.

겹치는 부분이 꽤 있지만 각 API 마다 요구하는 사항이 조금씩 달라서 3개의 컴포넌트를 만들었습니다.

네이버 로그인의 경우 CORS 때문에 우선 PROXY를 통해 임시방편 해놓은 상태인데 이것은 백엔드와 이야기를 해봐야할 것 같습니다.
네이버 로그인을 진행하려면 package.json에
"proxy": "https://nid.naver.com"을 추가하셔야 합니다.

google drive 프론트엔드 폴더에 개발 환경일 때의 redirect_url을 추가했습니다.
확인해보시고 env파일 수정 후 실행해주시면 됩니다.

@ChoiWonBeen
Copy link
Member

ChoiWonBeen commented Jul 11, 2023

RedirectURL이 전부 api.stage.jjbaksa... 로 되어있는 채라 App.tsx에 해당된 경로로 이동되지 않는데 env파일을 바꿔야할까요? 바꿔야한다면 드라이브 파일 업데이트 부탁드려요

@chaeseungyun
Copy link
Contributor Author

구글 드라이브 업데이트 했으니 모두 실행 가능할 겁니다!

Copy link
Member

@ChoiWonBeen ChoiWonBeen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네이버 서버 이슈 제외하곤 잘 되네요!
근데 구글로그인 도중에 "구글 로그인임ㅋ" 이거 살짝 보여서 지워주셔야할듯해요 ㅋㅋㅋ

Copy link
Contributor

@KimKyungYun KimKyungYun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인해보니 모두 잘 동작하는 것 같습니다! 고생하셨어요~

@chaeseungyun chaeseungyun merged commit b77edf2 into develop Jul 20, 2023
1 check passed
@chaeseungyun chaeseungyun deleted the feature/#98 branch July 20, 2023 07:43
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.

3 participants