Skip to content

Refactor/#174 Google Ads OAuth 로그인 성공 시 상태코드 및 응답 수정#175

Merged
jinnieusLab merged 2 commits into
developfrom
refactor/#174
Jul 10, 2026
Merged

Refactor/#174 Google Ads OAuth 로그인 성공 시 상태코드 및 응답 수정#175
jinnieusLab merged 2 commits into
developfrom
refactor/#174

Conversation

@jinnieusLab

Copy link
Copy Markdown
Collaborator

📌 관련 이슈

🚀 개요

이번 PR에서 변경된 핵심 내용을 요약해주세요.

Google Ads OAuth login 성공 시 상태코드를 200으로 변경 및 응답에 redirect url 함께 내리도록 수정하였습니다.

📄 작업 내용

구체적인 작업 내용을 설명해주세요.

  • GoogleAdsOAuthController 수정 [4e2bd7c]
    : 기존처럼 status를 FOUND(302)로 고정 x, ResponseEntity 응답에 redirect url 추가

📸 스크린샷 / 테스트 결과 (선택)

결과물 확인을 위한 사진이나 테스트 로그를 첨부해주세요.

image

기존

image

변경 후

✅ 체크리스트

  • 브랜치 전략(GitHub Flow)을 준수했나요?
  • 메서드 단위로 코드가 잘 쪼개져 있나요?
  • 테스트 통과 확인
  • 서버 실행 확인
  • API 동작 확인

🔍 리뷰 포인트 (Review Points)

  • 백엔드 측 테스트(Postman)에서는 상태 코드 302이어도 Google Ads OAuth 로그인 화면으로 이동하는 callback url이 응답으로 반환 되었으나, 프론트 테스트 환경에서는 302 응답과 함께 응답 대신 해당 Location으로 바로 이동되는 흐름 -> 하지만 이렇게 바로 이동시 CORS 에러 발생, 리다이렉트가 차단됨

해당 이유로 수정 후 바로 테스트로 반영 확인하였습니다! 바로 머지하겠습니다!

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

@jinnieusLab jinnieusLab self-assigned this Jul 10, 2026
@jinnieusLab jinnieusLab added the ♻️ Refactor 코드 구조 개선 label Jul 10, 2026
@jinnieusLab jinnieusLab merged commit a808c4f into develop Jul 10, 2026
1 check passed
@jinnieusLab jinnieusLab deleted the refactor/#174 branch July 10, 2026 10:15
@coderabbitai

coderabbitai Bot commented Jul 10, 2026

Copy link
Copy Markdown

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/#174

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ Refactor 코드 구조 개선

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor: Google Ads OAuth 로그인 성공 시 상태코드 및 응답 수정

1 participant