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/#18 #32

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Feature/#18 #32

wants to merge 17 commits into from

Conversation

yuri-0415
Copy link
Contributor

No description provided.

@gyusang gyusang linked an issue Jan 3, 2021 that may be closed by this pull request
@yuri-0415
Copy link
Contributor Author

@gyusang 상규님 주석부분 확인했습니다! 우선 serializer를 작성하면 될까요? social account에 대해 작성하면 될지 궁금합니다!

@gyusang
Copy link
Collaborator

gyusang commented Jan 4, 2021

@gyusang 상규님 주석부분 확인했습니다! 우선 serializer를 작성하면 될까요? social account에 대해 작성하면 될지 궁금합니다!

페이스북은 연동 안 해도 될까요?

@yuri-0415
Copy link
Contributor Author

@gyusang 상규님 주석부분 확인했습니다! 우선 serializer를 작성하면 될까요? social account에 대해 작성하면 될지 궁금합니다!

페이스북은 연동 안 해도 될까요?

그부분 확인해보겠습니다!

@yuri-0415
Copy link
Contributor Author

페이스북은 우선 allauth에서 제공하는 providers로 작성해두었습니다. 상규님처럼 Custom Facebook 작성해보려고 하는데 Custom Google은 어떤 목적으로 작성되었던 건지 다시 한번 설명해주실 수 있을까요? 😄

api_view = SocialLoginView()
api_view.dispatch(request, *args, **kwargs)
res = super(TokenOAuth2CallbackView, self).dispatch(request, *args, **kwargs)
if isinstance(res, HttpResponseRedirect) and res.url == settings.LOGIN_REDIRECT_URL:
Copy link
Collaborator

@gyusang gyusang Jan 6, 2021

Choose a reason for hiding this comment

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

페이스북은 우선 allauth에서 제공하는 providers로 작성해두었습니다. 상규님처럼 Custom Facebook 작성해보려고 하는데 Custom Google은 어떤 목적으로 작성되었던 건지 다시 한번 설명해주실 수 있을까요? 😄

@yuri-0415 Custom Google을 만든 것은, callback에서 로그인에 성공했을 때 response를 302 redirect 대신 200 ok로 바꾸고, 안에 user 정보와 토큰 정보를 포함하기 위함입니다.
또한 /accounts/google/login/을 요청했을 때에도 로그인 링크로 302 redirect되는 대신 url을 body에 포함한 200 ok가 됩니다. (API 문서를 참고해 주세요)

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.

Add Social Login
2 participants