Skip to content

bug(#75) : redirect-uri 환경변수 이름 변경#76

Merged
fixgramwork merged 1 commit intodevelopfrom
bug/#75
Sep 7, 2025
Merged

bug(#75) : redirect-uri 환경변수 이름 변경#76
fixgramwork merged 1 commit intodevelopfrom
bug/#75

Conversation

@Hgyeol
Copy link
Member

@Hgyeol Hgyeol commented Sep 7, 2025

[#75] 환경변수 이름 변경


📑 개요

환경변수 이름 변경


✅ 작업 내용

  • 환경변수 이름 변경

🔗 관련 이슈

Close #75


📌 체크리스트

  • 코드 컨벤션을 지켰나요?
  • 커밋 메시지 컨벤션을 지켰나요?
  • 테스트를 완료했나요?

Summary by CodeRabbit

  • Chores
    • OAuth2 Google 로그인 리디렉트 URI의 환경 변수 키를 GOOGLE_REDIRECT_URI에서 REDIRECT_URI로 변경했습니다. 배포 환경에서 해당 변수명을 업데이트해야 로그인 리디렉션이 정상 동작합니다.

@coderabbitai
Copy link

coderabbitai bot commented Sep 7, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Spring OAuth2 Google client configuration in application.yaml updates the redirect-uri environment variable reference from ${GOOGLE_REDIRECT_URI} to ${REDIRECT_URI}. No other configuration keys or public APIs are changed.

Changes

Cohort / File(s) Summary
Config: OAuth2 Google redirect URI
src/main/resources/application.yaml
Changed Spring OAuth2 client.registration.google.redirect-uri from ${GOOGLE_REDIRECT_URI} to ${REDIRECT_URI}.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Assessment against linked issues

Objective Addressed Explanation
환경변수 이름 변경으로 google_redirect_uri 미읽힘 버그 해결 (#75)

Assessment against linked issues: Out-of-scope changes

(없음)

Poem

새로 튄 URI, 귀가 쫑긋- 척 알아채고,
변수 이름 폴짝 바꿔 길을 바로 잡고,
리다이렉트 길 따라 달빛을 건너가네.
깡총, 깡총—설정도 정돈!
오늘 밤 배포엔 당근 한 개 더! 🥕🐇


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 24b67a5 and 2c5a908.

📒 Files selected for processing (1)
  • src/main/resources/application.yaml (1 hunks)
✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bug/#75

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 and usage tips.

@fixgramwork fixgramwork merged commit 4474ae2 into develop Sep 7, 2025
1 check was pending
@fixgramwork fixgramwork deleted the bug/#75 branch November 19, 2025 06:49
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.

[BUG] google_redirect_uri 환경변수 못 읽음

2 participants