Skip to content

Conversation

@mungyun
Copy link
Contributor

@mungyun mungyun commented Nov 13, 2024

📌 작업 내용

  • login페이지에서 구글의 간편 로그인 기능을 구현했습니다.
  • favorite페이지의 SSR 부분에서 프록시 서버를 거치는 부분을 제거하고 클라이언트에서 바로 linkbray 백엔드 서버로 get요청을 하게 수정했습니다.

@mungyun mungyun self-assigned this Nov 13, 2024
<div className="flex gap-4">
<Link href="https://accounts.google.com/o/oauth2/v2/auth?scope=openid%20profile%20email&response_type=token&redirect_uri=http://localhost:3000/google&client_id=1079911783112-7rg5ecp9ia9lorm7pit0m2nb2ti1rpt0.apps.googleusercontent.com">
<Link
href={`https://accounts.google.com/o/oauth2/v2/auth?scope=https://www.googleapis.com/auth/userinfo.profile&response_type=code&redirect_uri=http://localhost:3000/api/auth/sign-in/google&client_id=${process.env.GOOGLE_CLIENT_ID}`}
Copy link
Collaborator

Choose a reason for hiding this comment

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

GOOGLE_CLIENT_ID는 .env 파일에서 관리하는거죠? 한번 공유 주시면 좋겠습니다 : )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아이디 생성 과정에 대해서 말씀하시는거죠?? 팀 미팅때 공유해드리겠습니당~

Copy link
Collaborator

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

Oauth 인증 관련 흐름만 대략 알고있었는데 코드도 작성할게 은근 많아보이네요 수고하셨습니다 !

Copy link
Collaborator

@99minji 99minji left a comment

Choose a reason for hiding this comment

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

나중에 간편 로그인 구현할 때 많은 도움이 될 것 같습니닷! 감사합니다 👍

Comment on lines +15 to +19
.post("/api/auth/sign-up/google", {
name: "박문균",
token: accessToken,
redirectUri: "http://localhost:3000/google",
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

테스트 하기 위해 name 을 고정으로 넣어보신 건가욧!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 이 페이지는 없애야하는 데 깜빡했네여 merge하기 전에 수정해서 push할게욥

@mungyun mungyun closed this Nov 14, 2024
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.

5 participants