Conversation
개요이 PR은 Redis 캐싱을 좋아요 관련 작업 및 사용자 인증에 추가하고, PostgreSQL과 Redis 서비스를 위한 로컬 Docker Compose 구성을 제공합니다. 변경 사항
예상 코드 리뷰 노력🎯 3 (Moderate) | ⏱️ ~20분 관련 가능성 있는 PR
시 🐰
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/main/java/com/be/sportizebe/global/security/CustomUserDetailService.java (1)
17-23:⚠️ Potential issue | 🟠 MajoruserDetails 캐시 무효화 전략이 필요합니다.
UserDetails를 캐시하면서 사용자 정보 수정 시 캐시가 무효화되지 않아 이전 데이터로 계속 인증됩니다. 특히
uploadProfileImage()메서드가 User 엔티티를 수정하지만 캐시 정제 없이 진행되므로, RedisCacheConfig에서 userDetails에 대한 명시적 TTL 설정과 사용자 변경 흐름에서@CacheEvict(cacheNames = "userDetails", key = "#username")을 추가해야 합니다.
🤖 Fix all issues with AI agents
In `@src/main/java/com/be/sportizebe/domain/like/service/LikeServiceImpl.java`:
- Around line 56-57: The cache key in isLiked currently uses "#user.id" which
throws an NPE when `@AuthenticationPrincipal` User user is null for
unauthenticated calls (seen from getPostLikeStatus/getCommentLikeStatus); update
the `@Cacheable` key expression on isLiked to be null-safe (for example use a
ternary or null-check like "#user == null ? 'ANON' : `#user.id` + ':' +
`#targetType` + ':' + `#targetId`" or use SpEL null-safe operators) so that
unauthenticated requests produce a stable key instead of causing an exception.
🧹 Nitpick comments (3)
docker/docker-compose.local.yml (3)
1-1:version키는 Docker Compose V2에서 더 이상 사용되지 않습니다.Docker Compose V2(Compose Specification)부터
version키는 obsolete로 처리됩니다. 제거해도 무방하며, 유지해도 동작에는 영향이 없습니다.♻️ 제안된 수정
-version: "3.9" - services:
35-36: Redis healthcheck에서 비밀번호 경고가 발생할 수 있습니다.
-a password옵션을 명령줄에 직접 전달하면 Redis가Warning: Using a password with '-a' or '-u' option on the command line interface may not be safe경고를 출력합니다.REDISCLI_AUTH환경 변수를 사용하면 이 경고를 피할 수 있습니다.♻️ 경고 방지를 위한 제안
healthcheck: - test: [ "CMD", "redis-cli", "-a", "password", "PING" ] + test: [ "CMD-SHELL", "REDISCLI_AUTH=password redis-cli PING" ] interval: 10s timeout: 3s retries: 5
15-16: 포트가 모든 인터페이스에 바인딩됩니다.현재 포트 매핑이
0.0.0.0(모든 네트워크 인터페이스)에 바인딩됩니다. 로컬 개발 환경에서 외부 네트워크 노출을 방지하려면127.0.0.1로 제한하는 것을 고려해 주세요.🔒️ 로컬호스트 전용 바인딩 제안
ports: - - "5432:5432" + - "127.0.0.1:5432:5432"ports: - - "6379:6379" + - "127.0.0.1:6379:6379"Also applies to: 30-31
| @Cacheable(cacheNames = "likeStatus", key = "#user.id + ':' + #targetType + ':' + #targetId") | ||
| public boolean isLiked(User user, LikeTargetType targetType, Long targetId) { |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
cd /repo && find . -name "LikeServiceImpl.java" -type fRepository: Sportize/be
Length of output: 110
🏁 Script executed:
cd /repo && find . -name "*LikeController*.java" -type f 2>/dev/null || find . -name "*Like*Controller*.java" -type f 2>/dev/null | head -20Repository: Sportize/be
Length of output: 204
🏁 Script executed:
cd /repo && head -80 src/main/java/com/be/sportizebe/domain/like/service/LikeServiceImpl.javaRepository: Sportize/be
Length of output: 110
🏁 Script executed:
head -80 src/main/java/com/be/sportizebe/domain/like/service/LikeServiceImpl.javaRepository: Sportize/be
Length of output: 2534
🏁 Script executed:
cat src/main/java/com/be/sportizebe/domain/like/controller/LikeController.javaRepository: Sportize/be
Length of output: 3197
🏁 Script executed:
grep -n "isLiked" src/main/java/com/be/sportizebe/domain/like/service/LikeServiceImpl.javaRepository: Sportize/be
Length of output: 137
비로그인 접근 시 SpEL NPE 위험이 실제로 발생할 수 있습니다.
getPostLikeStatus와 getCommentLikeStatus 엔드포인트에서 @AuthenticationPrincipal User user는 인증이 없을 때 null을 주입합니다(required=true 미설정). 인증 없이 이 엔드포인트들에 접근 가능하므로, isLiked 메서드의 캐시 키 표현식 #user.id에서 NPE가 발생합니다.
null-safe 처리를 추가해 주세요:
💡 null-safe 처리 예시
- `@Cacheable`(cacheNames = "likeStatus", key = "#user.id + ':' + `#targetType` + ':' + `#targetId`")
+ `@Cacheable`(
+ cacheNames = "likeStatus",
+ key = "#user.id + ':' + `#targetType` + ':' + `#targetId`",
+ condition = "#user != null"
+ )
public boolean isLiked(User user, LikeTargetType targetType, Long targetId) {
+ if (user == null) {
+ return false;
+ }
return likeRepository.existsByUserAndTargetTypeAndTargetId(user, targetType, targetId);
}🤖 Prompt for AI Agents
In `@src/main/java/com/be/sportizebe/domain/like/service/LikeServiceImpl.java`
around lines 56 - 57, The cache key in isLiked currently uses "#user.id" which
throws an NPE when `@AuthenticationPrincipal` User user is null for
unauthenticated calls (seen from getPostLikeStatus/getCommentLikeStatus); update
the `@Cacheable` key expression on isLiked to be null-safe (for example use a
ternary or null-check like "#user == null ? 'ANON' : `#user.id` + ':' +
`#targetType` + ':' + `#targetId`" or use SpEL null-safe operators) so that
unauthenticated requests produce a stable key instead of causing an exception.
2 participants
#️⃣ Issue Number
📝 요약(Summary)
로그인/인증 시 불필요한 DB 조회를 줄였습니다.
• 로컬 개발 환경 구성을 위해 docker-compose.local.yml 설정을 추가했습니다.
🛠️ PR 유형
어떤 변경 사항이 있나요?
📸스크린샷 (선택)
💬 공유사항 to 리뷰어
✅ PR Checklist
PR이 다음 요구 사항을 충족하는지 확인하세요.
Summary by CodeRabbit
릴리스 노트
성능 개선
Chores