Skip to content

[#86] Google 인증 API를 NestJS로 이전한다 - #92

Merged
opficdev merged 18 commits into
developfrom
refactor/#86
Aug 6, 2026
Merged

[#86] Google 인증 API를 NestJS로 이전한다#92
opficdev merged 18 commits into
developfrom
refactor/#86

Conversation

@opficdev

@opficdev opficdev commented Aug 5, 2026

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • 기존 Functions의 Google 인증과 계정 관리 API를 NestJS로 이전
  • 기존 OAuth, Firebase Auth, Firestore, HTTP 응답 계약 유지
  • staging의 Google 인증 요청을 Cloud Run http-api로 전환
  • Functions와 Production 배포 제외

📝 작업 내용

📌 요약

  • Google custom token 발급, 계정 연결·해제, grant 폐기 API 구현
  • 공개 인증 경계를 위한 @Public()FirebaseAuthGuard 연동
  • Google OAuth Client, Repository, Service, Controller와 관련 시험 추가
  • staging Hosting에 /api/auth/google/** Cloud Run rewrite 추가
  • Cloud Run 로그의 단일 줄 JSON 구조화
  • 이전된 Functions route·handler·관련 시험 정리
  • Cloud Run과 Hosting staging 배포 문서 갱신

🔍 상세

  • POST /api/auth/google/authorization-code/custom-token 공개 경로 이전

  • PUT /api/auth/google/authorization-code/account-link 인증 경로 이전

  • DELETE /api/auth/google/account-link 인증 경로 이전

  • DELETE /api/auth/google/access-token 인증 경로 이전

  • Google ID token의 서명, aud, iss, 만료 검증 유지

  • Firebase 사용자 결정, provider 연결, credential 저장과 refresh token 보존 정책 유지

  • account-link claim 획득과 해제, credential 정리 순서 유지

  • 이미 폐기된 token 처리와 기존 오류 응답 계약 유지

  • GOOGLE_OAUTH_CONFIG Secret의 Cloud Run 주입 구성 추가

  • 사용자 삭제에서 계속 사용하는 Google grant 폐기 구현 유지

  • 이전 대상 외 기존 Functions REST 경로 유지

  • 색상 제어 문자와 물리적인 줄바꿈 없이 한 항목의 jsonPayload로 수집되는 Cloud Run 로그 구성

  • nestjs-api lint와 build 통과

  • nestjs-api 단위 시험 26 suites, 203 tests 통과

  • nestjs-api E2E 시험 4 suites, 46 tests 통과

  • functions build와 전체 시험 통과

  • git diff --check 통과

  • staging Cloud Run revision http-api-00004-m4wReady 상태와 트래픽 100% 확인

  • staging Hosting 배포와 Google 경로의 http-api 전달 확인

  • 앱에서 Google 로그인, 로그아웃, 계정 연결·해제, 회원탈퇴 동작 확인

  • Cloud Logging의 단일 jsonPayload 수집과 기존 여러 줄 출력 제거 확인

  • Production 배포 미실행

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Aug 5, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 115447d9b9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread nestjs-api/src/google/google-provider.repository.ts Outdated
Comment thread nestjs-api/src/google/google-provider.repository.ts Outdated
Comment thread firebase.json

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e3b6d7a153

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread nestjs-api/src/google/google-provider.repository.ts
@opficdev
opficdev merged commit 69ad136 into develop Aug 6, 2026
4 checks passed
@opficdev
opficdev deleted the refactor/#86 branch August 6, 2026 00:51
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.

Google 인증 API를 NestJS로 이전한다

1 participant