ddd#80
Conversation
|
Caution Review failedThe pull request is closed. Walkthrough피드 관련 DTO(enum 포함)와 컨트롤러/서비스에 페이징 및 통합 FeedResponse가 도입되고, application.yml에 Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant FeedController
participant FeedService
participant DiaryRepo
participant AvatarRepo
Client->>FeedController: GET /feed?filter=X&page=P&size=S
FeedController->>FeedService: getFeed(currentUserUuid, filter, P, S)
FeedService->>DiaryRepo: findDiaries(candidatePageable)
FeedService->>AvatarRepo: findAvatarPosts(candidatePageable)
DiaryRepo-->>FeedService: List<Diary>
AvatarRepo-->>FeedService: List<AvatarPost>
FeedService->>FeedService: map to FeedResponse streams\nconcat streams -> sort by createdAt desc
FeedService->>FeedService: slice start=P*S, end=min(start+S, total)
FeedService-->>FeedController: List<FeedResponse> (paged)
FeedController-->>Client: 200 OK { data: [...] }
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (4)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
🚨 PR 본문이 비어있습니다! 아래 템플릿을 복사하여 PR 내용을 작성해주세요. 📝 개요
💻 작업 내용
✅ PR 체크리스트
🔗 관련 이슈
스크린샷 (선택)
|
* refactor: 피드 조회 api dto 수정 * spotless apply * spotless apply * spotless apply * spotless apply
…/github.com/Const-Project/MainBE into refactor/diary-with-comment-and-remove-item
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/main/resources/application.yml (2)
78-82: Swagger 서버 설명 변경은 무해. 다만 URL/설명을 환경 변수로 외부화 고려
description: Production Server형식 변경은 의미상 문제 없습니다. 운영/스테이징/개발 환경 분리를 깔끔히 하려면 URL과 설명을 프로파일별 환경 변수로 외부화하면 유지보수성이 좋아집니다. 예:${SWAGGER_SERVER_URL},${SWAGGER_SERVER_DESC}. 프록시 헤더 전략 변경으로 Swagger의 서버 URL/스킴이 프록시 기준으로 노출되는지 함께 점검하세요(절대 URL 생성 시 예상과 다른 스킴/호스트가 보이는지).가능한 개선 예시:
springdoc: swagger-ui: servers: - url: ${SWAGGER_SERVER_URL:https://api.napulnapul.com} description: ${SWAGGER_SERVER_DESC:Production Server}
82-82: 파일 끝 개행(Newline at EOF) 추가 필요lint 경고가 있으므로 파일 끝에 개행을 추가해 주세요.
적용 예시:
- description: Production Server + description: Production Server +
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
src/main/resources/application.yml(2 hunks)
🧰 Additional context used
🪛 YAMLlint (1.37.1)
src/main/resources/application.yml
[error] 82-82: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (1)
src/main/resources/application.yml (1)
1-3: server.forward-headers-strategy 설정 검증 요청다음 사항을 확인하였습니다. 추가 검증 및 인프라 환경 점검이 필요합니다:
- Spring Boot 버전 호환성
• build.gradle에 선언된org.springframework.boot버전이 3.5.3으로,server.forward-headers-strategy: framework를 지원합니다.- 중복 헤더 처리기 등록 여부
• 코드베이스 내에ForwardedHeaderFilter수동 등록이나 TomcatRemoteIpValve설정 (server.tomcat.remoteip)이 없습니다.- 프록시 헤더 세탁(sanitization) 및 재작성 규칙
• 코드 외부의 인프라(Nginx, Ingress 등)에서 신뢰 가능한 프록시만Forwarded/X-Forwarded-*헤더를 허용하도록 구성되어 있는지 운영팀과 함께 확인해주세요.- 프로파일 적용 범위
•src/main/resources/application.yml(1–3행)의 설정이 모든 프로파일에 적용되므로, 로컬 개발 환경 영향이나 의도치 않은 URL 재작성 방지를 위해 프로덕션 전용 프로파일(prod)로 제한 적용을 권장합니다.[src/main/resources/application.yml:1–3]
3 participants
Summary by CodeRabbit