[woo6629058]

#️⃣ 연관된 이슈

#11

📝 작업 내용

백엔드에서 관리하던 기본 프로필 이미지 설정을 제거하고, 프로필 이미지가 없는 경우 profileImageUrl을 null로 반환하도록 변경했습니다.

• 기본 프로필 이미지는 프론트엔드에서 정적 이미지로 처리하며, 사용자가 프로필 이미지를 등록한 경우에는 기존처럼 S3 Presigned GET URL을 반환합니다.

변경 사항

• ProfileService의 기본 프로필 이미지 URL 주입 및 대체 로직 제거
• 프로필 이미지 Object Key가 없으면 profileImageUrl: null 반환
• 프로필 이미지 Object Key가 있으면 기존 Presigned GET URL 반환 유지
• app.profile.default-image-url 설정 제거
• .env.example에서 DEFAULT_PROFILE_IMAGE_URL 제거
• README의 관련 환경변수 설명 제거
• 프로필 이미지 유무에 따른 응답 단위 테스트 추가

스크린샷 (Swagger 등)

💬 리뷰 요구사항

PR 전 필수 체크리스트

• 관련 이슈를 연결했습니다.
• 로컬에서 정상 동작을 확인했습니다.
• 필요한 테스트를 수행했습니다.
• Swagger/API 명세 변경이 필요하지 않음을 확인했습니다.
• 불필요한 로그 또는 주석을 제거했습니다.
• 민감 정보가 포함되지 않았는지 확인했습니다.

테스트 결과

• 단위 테스트
• 통합 테스트
• Swagger 테스트
• Postman 테스트
• 직접 실행 확인

Summary by CodeRabbit

• 변경 사항

  • 기본 프로필 이미지 URL 설정 및 관련 환경 변수 안내가 제거되었습니다.
  • 프로필 이미지가 등록되지 않은 경우 기본 이미지 대신 이미지 URL이 제공되지 않습니다.
  • 프로필 이미지가 등록된 경우에는 기존과 동일하게 다운로드 URL이 제공됩니다.

• 테스트

  • 프로필 이미지가 없거나 등록된 경우의 프로필 조회 동작을 검증하는 테스트가 추가되었습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: e9d47841-e407-4118-a855-7dfbd036f262

📥 Commits

Reviewing files that changed from the base of the PR and between 7a20b57 and cdf858a.

📒 Files selected for processing (5)

• .env.example
• README.md
• src/main/java/com/quespot/domain/user/service/ProfileService.java
• src/main/resources/application.yaml
• src/test/java/com/quespot/domain/user/service/ProfileServiceTest.java

💤 Files with no reviewable changes (3)

• README.md
• src/main/resources/application.yaml
• .env.example

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

기본 프로필 이미지 URL 환경 변수와 설정을 제거했습니다. ProfileService는 이미지 객체 키가 없으면 null을 반환하고, 객체 키가 있으면 기존 presigned 다운로드 URL을 반환합니다. 두 동작을 검증하는 테스트를 추가했습니다.

Changes

프로필 이미지 기본값 제거

Layer / File(s)	Summary
프로필 이미지 처리 변경 .env.example, README.md, src/main/java/com/quespot/domain/user/service/ProfileService.java, src/main/resources/application.yaml	기본 프로필 이미지 URL 설정과 생성자 인자를 제거했습니다. 객체 키가 없거나 공백이면 null을 반환하고, 유효한 객체 키에는 presigned 다운로드 URL을 생성합니다.
프로필 이미지 동작 테스트 src/test/java/com/quespot/domain/user/service/ProfileServiceTest.java	객체 키가 없을 때 fileService를 호출하지 않고 null을 반환하는 동작과, 객체 키가 있을 때 presigned URL을 반환하는 동작을 검증합니다.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to cdf85

The change cleanly removes the default image fallback while preserving presigned URLs for stored images, with no unresolved merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 2 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 백엔드 기본 프로필 이미지 설정 제거라는 주요 변경 사항을 정확하게 설명합니다. 제목은 간결하고 구체적입니다.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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