Skip to content

Fix/fix 단순 수정#120

Merged
CheatIsKey merged 27 commits into
developfrom
fix/fix
Apr 27, 2026
Merged

Fix/fix 단순 수정#120
CheatIsKey merged 27 commits into
developfrom
fix/fix

Conversation

@munhyerin22
Copy link
Copy Markdown
Contributor

OAuth제거 및 하드코딩 되어있는 이름 변경 등 수정함

@github-actions
Copy link
Copy Markdown

🤖 AI 코드리뷰

리더보드 — 현재 4위 (리뷰 101건) | 전체 8개 팀 참여 중
업데이트 확인 — 리뷰가 질문을 던지다 — 생각해보기

📋 과제 요구사항 준수 여부

  • 필수 기능 (동시성 제어 / 캐싱 / 검색): 이 PR에서 확인할 수 없음. (기존에 구현되었는지는 전체 코드를 봐야 알 수 있음)
  • 사용 금지 기술: 이 diff 내에서 Redisson 사용 감지되지 않음.

🔴 보안 / 성능 주의

  • 하드코딩된 더미 데이터: checkout.html'르탄이', 'example@gmail.com', '01012345678' 같은 값이 하드코딩되어 있습니다. PR 설명은 "단순 수정"이지만, Thymeleaf 서버 사이드 렌더링을 사용하면서 정적 텍스트로 기본값을 박아두는 것은 좋은 패턴이 아닙니다. 테스트용이라면 test 프로필에서만 로드되도록 분리하세요.
  • 결제 및 OAuth2 기능 전면 삭제: PortOne 결제 연동 설정 및 OAuth2 클라이언트 의존성이 완전히 제거되었습니다. 실수로 누락된 것인지 의도된 롤백인지 PR 설명이나 커밋 메시지로 명확한 맥락이 필요합니다. 운영 환경에 머지될 경우 결제 기능이 즉시 마비됩니다.

🤔 생각해보기

결제 게이트웨이(PortOne) 연동 설정을 의존성(build.gradle)부터 설정 값(application.yml), 그리고 뷰 템플릿(checkout.html)에 이르기까지 완벽하게 지워버렸습니다. 만약 이 브랜치가 실서버(운영 환경)에 배포된다면, 현재 결제를 시도하고 있는 사용자들의 세션과 트랜잭션은 어떤 방식으로 실패하게 될까요? 장애 발생 시 이 결제 기능의 부재가 비즈니스에 미치는 임팩트를 어떻게 통제할 수 있을까요?

💬 이 질문에 대해 궁금한 점이 있으면 코멘트에 @sparta 를 남겨보세요!
예: @sparta 결제 모듈 연동 해제를 안전하게 롤백하려면 어떤 배포 전략을 써야 하나요?


AI 리뷰는 참고용입니다. 최종 판단은 팀원이 직접 합니다.

@CheatIsKey CheatIsKey merged commit 8b8baee into develop Apr 27, 2026
2 checks passed
@CheatIsKey CheatIsKey deleted the fix/fix branch April 27, 2026 15:34
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Allday-Project-Board Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants