🔧Settings: 프론트 배포 도메인 CORS 추가 #20

uni-j-uni · 2025-12-15T14:45:26Z

Summary by CodeRabbit

릴리스 노트

버그 수정
- 카테고리 필터링이 선택 사항으로 변경되어 더 유연한 게시물 검색이 가능합니다.
개선 사항
- 게시물 조회 로직이 최적화되어 더 효율적으로 작동합니다.
- OAuth2 로그인 후 리다이렉트 경로가 업데이트되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

…ture/chat

coderabbitai · 2025-12-15T14:45:35Z

Walkthrough

Post 조회 기능을 리팩토링하여 category 파라미터를 선택사항으로 변경하고, PostRepository 메서드 시그니처를 정리하며, 카테고리 필터링 로직을 PostCategory 열거형 기반으로 개선했습니다. OAuth2 리다이렉트 URL을 프로덕션 환경으로 업데이트했습니다.

Changes

Cohort / File(s)	변경 요약
Post 조회 파라미터 최적화 `src/main/java/com/sku/refit/domain/post/controller/PostController.java`	category `@RequestParam을` required = false로 변경하여 선택사항 파라미터로 전환; lastPostId 파라미터 설명 조정
Repository 메서드 시그니처 정리 `src/main/java/com/sku/refit/domain/post/repository/PostRepository.java`	String 기반 카테고리 필터링 메서드 2개 제거 (findByPostCategoryContainingAndIdLessThan, findAllByUser_IdAndIdLessThan); PostCategory 열거형 기반 및 id 기반 조회 메서드 2개 추가 (findByIdLessThan, findByPostCategoryAndIdLessThan)
Post 조회 로직 개선 `src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java`	null/blank 카테고리 처리 로직 추가; PostCategory.valueOf를 통한 카테고리 검증 구현; lastPostId 유무에 따른 조건부 쿼리 메서드 선택; 커서 계산 방식 변경 (getLast 헬퍼 제거 후 직접 인덱스 접근)
프로덕션 환경 구성 `src/main/java/com/sku/refit/global/security/OAuth2LoginSuccessHandler.java`	OAuth2 로그인 성공 후 리다이렉트 URL을 http://localhost:3000에서 https://refit-lab.vercel.app으로 변경
서브모듈 참조 업데이트 `src/main/resources`	서브모듈 커밋 참조 업데이트 (메타데이터 변경만, 기능 변경 없음)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12분

주의 대상:
- PostServiceImpl의 카테고리 검증 로직과 조건부 쿼리 메서드 선택 흐름 확인 필요
- PostRepository 메서드 변경에 따른 호출부 사이드 이펙트 검증 (PostServiceImpl 외 다른 호출처 확인)
- OAuth2 리다이렉트 URL 변경의 보안 및 CORS 설정 영향 범위 확인

Possibly related PRs

♻️Refactor: 커뮤니티 게시글 전체 조회 가능하도록 수정 #19: PostController, PostRepository, PostServiceImpl의 동일한 메서드 시그니처 변경 및 카테고리 필터링 로직 수정으로 직접적으로 연관됨
Feature/community #2: Post 조회 및 카테고리 기반 필터링 기능 구현의 Repository 메서드 및 service 레이어 로직과 겹침
♻️Refactor: 게시글 좋아요 기능 추가 및 댓글 수, 좋아요 수 조회 추가 #14: getPostsByCategory 메서드 및 PostController의 게시물 목록 조회 흐름 수정으로 연관됨

Suggested labels

🟠 priority: medium, refactor, feature

Poem

🐰 카테고리 선택은 이제 자유로워,
null도 환영하는 유연한 쿼리,
PostCategory는 정확하게,
프로덕션으로 도약하는 리다이렉트!
코드는 정리되고, 기능은 뽀족해진 오늘. ✨

Pre-merge checks and finishing touches

❌ Failed checks (3 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	PR 제목이 OAuth2 리다이렉트 URL 변경만 다루고 있지만, 실제 변경사항에는 PostController 파라미터 수정, PostRepository 메서드 교체, PostServiceImpl 로직 변경 등이 포함되어 있어 주요 변경사항을 제대로 반영하지 못했습니다.	PR 제목을 변경사항 전체를 포괄하는 내용으로 수정하거나, 다양한 변경사항을 처리하는 여러 PR로 분리하는 것을 권장합니다.
Description check	⚠️ Warning	저장소의 필수 템플릿을 완전히 따르지 않았으며, 구현 내용, 테스트 방법, 추가 고려사항 등 필수 섹션이 모두 누락되어 있습니다.	제공된 템플릿에 따라 새로운 기능, 개발 상세, 테스트 방법, 추가 고려사항, 관련 문서/이슈 섹션을 작성하세요.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feature/chat

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.}

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6a5b6e3 and 7f8dd33.

📒 Files selected for processing (5)

src/main/java/com/sku/refit/domain/post/controller/PostController.java (1 hunks)
src/main/java/com/sku/refit/domain/post/repository/PostRepository.java (1 hunks)
src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java (2 hunks)
src/main/java/com/sku/refit/global/security/OAuth2LoginSuccessHandler.java (1 hunks)
src/main/resources (1 hunks)

🔇 Additional comments (5)

src/main/resources (1)

1-1: 서브모듈 변경사항 검증 필요

서브모듈 커밋 포인터만 업데이트되었으며, 실제 설정 파일의 변경 내용을 확인할 수 없습니다. PR 제목에서 언급한 CORS 설정이 올바르게 추가되었는지 검증이 필요합니다.

다음을 확인해주세요:

서브모듈 내용 검증: e9738e52... 에서 b18f530... 로의 변경에서 실제 어떤 설정 파일이 수정되었는지 확인

CORS 설정: 프론트엔드 배포 도메인이 spring.web.cors.allowed-origins 또는 WebMvcConfigurer 등에 올바르게 추가되었는지 확인

OAuth2 리다이렉트: PR 제목과 요약의 범위 불일치 - 실제로는 Post 조회 기능 리팩토링과 OAuth2 리다이렉트 URL 변경도 포함되어 있는 것으로 보임

보안: CORS 설정에서 허용된 출처가 프로덕션 환경에 적절한지 확인

기대되는 결과:

CORS allowedOrigins에 올바른 배포 도메인 포함

자격증명(credentials)이 필요한 경우 allowCredentials = true 설정 확인

HttpMethod가 필요한 범위로 제한되어 있는지 확인

src/main/java/com/sku/refit/domain/post/controller/PostController.java (1)

71-77: 카테고리 파라미터를 선택사항으로 변경

카테고리를 선택사항으로 변경하여 전체 게시글 조회를 지원하도록 개선된 점은 좋습니다. 서비스 레이어의 로직과도 일관성 있게 연동됩니다.

참고: 이 변경사항은 PR 제목("프론트 배포 도메인 CORS 추가")과는 직접적인 연관이 없어 보입니다. PR의 범위를 명확히 하는 것이 좋을 것 같습니다.

src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java (1)

215-215: 커서 계산 방식 변경 확인

posts.get(posts.size() - 1)로 직접 인덱스 접근하도록 변경된 것은 간결하며, 위의 posts.isEmpty() 체크로 인해 IndexOutOfBoundsException이 발생하지 않으므로 안전합니다.

src/main/java/com/sku/refit/domain/post/repository/PostRepository.java (1)

21-23: 새로운 리포지토리 메서드 추가 확인

커서 기반 페이지네이션을 지원하기 위한 새로운 메서드들이 적절하게 추가되었습니다:

findByIdLessThan: 카테고리 필터 없이 ID 기반 커서 페이징

findByPostCategoryAndIdLessThan: 카테고리 필터와 ID 기반 커서 페이징

메서드 시그니처가 Spring Data JPA 명명 규칙에 맞으며, 서비스 레이어의 리팩토링된 로직과 일관성 있게 연동됩니다.

src/main/java/com/sku/refit/global/security/OAuth2LoginSuccessHandler.java (1)

71-71: CORS 설정이 프로덕션 도메인을 허용하는지 확인 필요

OAuth2 인증 성공 후 https://refit-lab.vercel.app로 리다이렉트하는 코드는 적절합니다. 다만 CorsConfig.java에서 CORS 설정이 ${cors.allowed-origins} 환경 변수로 외부화되어 있기 때문에, 실제 배포 환경에서 이 도메인이 설정되어 있는지 확인이 필수입니다. 코드베이스 내 설정 파일에서는 실제 허용 도메인 값을 확인할 수 없으므로, 배포 환경 변수나 외부 설정 서버에서 https://refit-lab.vercel.app이 cors.allowed-origins에 포함되어 있는지 검증해 주세요.

coderabbitai · 2025-12-15T14:48:31Z

src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java

+    if (category == null || category.isBlank()) {
+      if (lastPostId == null) {
+        posts = postRepository.findAll(pageable).getContent();
+      } else {
+        posts = postRepository.findByIdLessThan(lastPostId, pageable).getContent();
+      }
    } else {
-      posts =
-          postRepository
-              .findByPostCategoryContainingAndIdLessThan(category, lastPostId, pageable)
-              .getContent();
+      PostCategory postCategory;
+      try {
+        postCategory = PostCategory.valueOf(category);
+      } catch (IllegalArgumentException e) {
+        throw new CustomException(PostErrorCode.INVALID_CATEGORY);
+      }
+
+      if (lastPostId == null) {
+        posts = postRepository.findByPostCategory(postCategory, pageable).getContent();
+      } else {
+        posts =
+            postRepository
+                .findByPostCategoryAndIdLessThan(postCategory, lastPostId, pageable)
+                .getContent();
+      }
    }


⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash # Find PostService test files fd "PostService.*Test\.java$|PostService.*Spec\.java$" --type f # Also search more broadly for test files fd ".*Test\.java$" --type f | grep -i post | head -20

Repository: refit-lab/refit-be

Length of output: 44

🏁 Script executed:

# Search for getPostsByCategory tests rg -n "getPostsByCategory|getPostByCategory" --type java -g "*Test.java" -A 5 -B 2

Repository: refit-lab/refit-be

Length of output: 210

🏁 Script executed:

# Let's also check the structure of the src directory git ls-files | grep -E "src/(test|main)" | head -30

Repository: refit-lab/refit-be

Length of output: 2189

🏁 Script executed:

#!/bin/bash # Search for all Test.java files without filters find . -name "*Test.java" -o -name "*Spec.java" 2>/dev/null | head -50

Repository: refit-lab/refit-be

Length of output: 44

🏁 Script executed:

# Check if test directory exists ls -la | grep -i test find . -maxdepth 2 -type d -name "*test*" 2>/dev/null

Repository: refit-lab/refit-be

Length of output: 72

🏁 Script executed:

# Get a complete picture of repository structure git ls-files | cut -d'/' -f1,2 | sort | uniq -c | head -20

Repository: refit-lab/refit-be

Length of output: 500

🏁 Script executed:

# Search for PostService in any Java file to understand the codebase better rg -n "class PostService" --type java

Repository: refit-lab/refit-be

Length of output: 187

🏁 Script executed:

#!/bin/bash # List all files in src/test directory find src/test -type f 2>/dev/null | head -50

Repository: refit-lab/refit-be

Length of output: 116

🏁 Script executed:

# List directory structure of src/test ls -la src/test 2>/dev/null find src/test -type d 2>/dev/null

Repository: refit-lab/refit-be

Length of output: 311

🏁 Script executed:

# Search for any Java files in src/test git ls-files | grep "src/test"

Repository: refit-lab/refit-be

Length of output: 116

🏁 Script executed:

# Search for PostService references in entire codebase including test rg -l "PostService" --type java | head -20

Repository: refit-lab/refit-be

Length of output: 270

🏁 Script executed:

#!/bin/bash # Check the only test file to see if it tests PostService cat src/test/java/com/sku/refit/RefitApplicationTests.java

Repository: refit-lab/refit-be

Length of output: 302

🏁 Script executed:

# Check the PostService interface to see the actual method name rg -n "getPostsByCategory|getPostByCategory|getPost" src/main/java/com/sku/refit/domain/post/service/PostService.java -A 3

Repository: refit-lab/refit-be

Length of output: 276

🏁 Script executed:

# Check the full PostServiceImpl around the reviewed lines sed -n '140,190p' src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java | cat -n

Repository: refit-lab/refit-be

Length of output: 2042

getPostsByCategory 메서드에 대한 테스트 케이스 작성 필요

카테고리 필터링 로직 리팩토링으로 타입 안전성이 개선되었습니다. 다만 현재 이 메서드에 대한 테스트 커버리지가 전혀 없습니다. 다음 모든 실행 경로에 대한 테스트 케이스를 추가해야 합니다:

category가 null/blank이고 lastPostId가 null인 경우

category가 null/blank이고 lastPostId가 존재하는 경우

유효한 category이고 lastPostId가 null인 경우

유효한 category이고 lastPostId가 존재하는 경우

잘못된 category 형식으로 CustomException(PostErrorCode.INVALID_CATEGORY) 발생하는 경우

🤖 Prompt for AI Agents

In src/main/java/com/sku/refit/domain/post/service/PostServiceImpl.java around lines 158 to 180, there are five untested execution paths in getPostsByCategory; add unit tests that cover: (1) category null/blank with lastPostId null, (2) category null/blank with lastPostId present, (3) valid category with lastPostId null, (4) valid category with lastPostId present, and (5) invalid category string that throws CustomException(PostErrorCode.INVALID_CATEGORY). Implement tests using your project’s test framework (JUnit) and mocking (Mockito) to stub postRepository responses (return Page or List as expected) and verify the service returns the repository results and/or throws the expected exception; assert repository method selection (verify correct repository method called for each scenario) and validate returned content or the thrown CustomException with the correct error code.

uni-j-uni added 13 commits December 15, 2025 15:17

♻️Refactor: 게시글 카테고리 추가

2b552b5

➕Dependency: WebSocket 의존성 추가

35dc1b9

♻️Refactor: 교환 게시글 식별자 추가 및 로직 수정

920a8db

✨Feat: 웹소켓 기반 채팅 기능 구현

255265e

Merge branch 'main' into feature/chat

67b5775

♻️Refactor: 게시글 쿼리 메소드 오류 수정

59f12f0

♻️Refactor: 코드 리뷰 기반 수정사항 반영

e1b1a72

♻️Refactor: 코드 리뷰 기반 수정사항 반영

93e6daf

Merge branch 'main' of https://github.com/refit-lab/refit-be into fea…

f7e9f5c

…ture/chat

♻️Refactor: 커뮤니티 게시글 전체 조회 가능하도록 수정

1d06f3c

🔧Settings: 프론트 배포 도메인 CORS 추가

5ab42e4

🔧Settings: 프론트 배포 도메인 CORS 추가

284bc71

🔧Settings: 프론트 배포 도메인 CORS 추가

7f8dd33

uni-j-uni self-assigned this Dec 15, 2025

coderabbitai bot reviewed Dec 15, 2025

View reviewed changes

uni-j-uni merged commit 33d1841 into main Dec 15, 2025
3 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

🔧Settings: 프론트 배포 도메인 CORS 추가 #20

🔧Settings: 프론트 배포 도메인 CORS 추가 #20

Uh oh!

uni-j-uni commented Dec 15, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Dec 15, 2025 •

edited

Loading

Uh oh!

coderabbitai bot left a comment

Uh oh!

coderabbitai bot Dec 15, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

🔧Settings: 프론트 배포 도메인 CORS 추가 #20

🔧Settings: 프론트 배포 도메인 CORS 추가 #20

Uh oh!

Conversation

uni-j-uni commented Dec 15, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai bot commented Dec 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Suggested labels

Poem

Pre-merge checks and finishing touches

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai bot Dec 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

uni-j-uni commented Dec 15, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 15, 2025 •

edited

Loading