📈[분석추가] &⚡️[성능 향상] Prometheus & Grafana 모니터링 환경 구축 & 성능 향상 by wjkim9 · Pull Request #101 · News-Deliver/Server

wjkim9 · 2025-07-18T02:37:52Z

Grafana로 Prometheus 데이터 시각화

SpringBoot 서버와 Redis 서버에 대한 모니터링 세팅 구현

Redis-cache 서버를 활용하여 로드 시간 개선

API 호출 시 DB 접속 후 반환> 608ms
API 호출 시 캐시에서 반환 > 1ms

항목	기존 시간	캐시 이후	성능 향상률
DB 조회	608ms	1ms	99.84%
ES 조회	189ms	0ms	100%

DB 쿼리 개선 및 인덱스 추가

중복 제거 로직 수행 시 SQL에 복합 인덱스를 적용받을 수 있게끔 쿼리를 변경
처리 시간 개선

Summary by CodeRabbit

신규 기능
- Redis 기반 캐싱 기능이 도입되어 인기 토픽 및 뉴스 리스트 조회 성능이 향상되었습니다.
- Prometheus 및 Grafana를 통한 서비스 모니터링이 지원됩니다.
- Redis Exporter 서비스가 추가되어 Redis 상태 모니터링이 가능합니다.
버그 수정
- 뉴스 중복 삭제 및 "[속보]" 태그 뉴스 삭제 로직의 날짜 필터가 더 정확하게 개선되었습니다.
성능 개선
- 데이터베이스 뉴스 테이블에 복합 인덱스가 추가되어 검색 속도가 향상되었습니다.
설정
- 캐시 및 모니터링 관련 환경설정이 추가 및 개선되었습니다.
기타
- 설정 저장 시 허용 개수 초과 예외가 추가되었습니다.
- 일부 경로의 JWT 인증 제외 정책이 변경되었습니다.

코드 상의 FIXME 리스트 수행할 것.

로직 추가 구현중

IntelliJ 자동 구성 폴더 추가

Prometheus를 통해 SpringBoot 서버와 Redis 서버 데이터 수집하여, Grafana로 대시보드 구현(시각화)

DB및 ElasticSearch 서버에 대한 접근 횟수를 줄이고 성능을 비약적으로 향상시킴

중복 제거 로직에서 SQL을 INDEX를 적용받을 수 있도록 수정 중복 제거 시 복합 INDEX를 참조하여 성능이 향상 됨

coderabbitai · 2025-07-18T02:37:58Z

Walkthrough

이번 변경 사항은 Redis 기반 캐시 도입, Prometheus 및 Grafana 모니터링 인프라 추가, 뉴스 데이터베이스 인덱스 최적화, 일부 비즈니스 로직 개선, 그리고 캐시 및 모니터링 관련 설정 파일 추가 및 수정이 포함됩니다. 또한, JWT 필터 예외 경로와 중복 뉴스 삭제 쿼리 로직이 조정되었습니다.

Changes

파일/경로 그룹	변경 요약
.gitignore	`.idea` 디렉토리 무시 항목 추가
SpringBoot/build.gradle, SpringBoot/src/main/resources/application.properties	Prometheus, Micrometer, Redis 캐시 관련 의존성 및 설정 추가, 모니터링 엔드포인트 및 태그 설정
SpringBoot/database/init.sql	뉴스 테이블에 복합 인덱스 2종 추가 및 주석 보강
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/HotTopic_Description	"## 기능 설명" 헤더 뒤에 빈 줄 추가 (포맷팅 변경)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java	Redis 캐시 적용, 명시적 생성자 추가, 캐시 키/만료 로직 및 로깅 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Controller/SettingController.java	컨트롤러 메서드 반환 타입을 `ResponseEntity`로 명시화하고, 삭제 메서드의 예외 처리 제거 및 간소화
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java	사용자 조회 및 설정 개수 초과 시 예외 처리 추가, 저장/수정/삭제 메서드 간소화, 설정 개수 제한 검사 메서드 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Config/RedisConfig.java	캐시용 RedisConnectionFactory, RedisTemplate, ObjectMapper 빈 추가 및 기존 세션 Redis와 분리
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/ErrorCode.java	`SETTING_LIMIT_EXCEEDED` 에러 코드 추가
SpringBoot/src/main/java/Baemin/News_Deliver/Global/JWT/JwtAuthenticationFilter.java	JWT 필터 예외 경로에서 `/api/auth/`, `/` 제거, 정적 리소스 및 OAuth 관련 경로만 예외 처리
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/Batch/listener/BatchJobCompletionListener.java	뉴스 중복 삭제 쿼리의 날짜 조건 변경 및 "[속보]" 포함 뉴스도 삭제하도록 확장
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/service/NewsEsService.java	뉴스 적재 쿼리의 날짜 필터를 범위 조건으로 변경
SpringBoot/src/main/java/Baemin/News_Deliver/NewsDeliverApplication.java	`@EnableCaching` 애노테이션 추가
docker-compose.yml, monitoring/prometheus.yml	Prometheus, Grafana, Redis Exporter 3종 서비스 추가 및 볼륨/네트워크 설정, Prometheus 스크랩 타겟 구성
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/GlobalExceptionHandler.java	`@RestControllerAdvice` 애노테이션 추가 및 SettingException 핸들러 코드 스타일 수정

Sequence Diagram(s)

sequenceDiagram
    participant Client
    participant HotTopicService
    participant RedisCache
    participant DB
    participant ES

    Client->>HotTopicService: getHotTopicList()
    HotTopicService->>RedisCache: 캐시 조회(hottopic:daily)
    alt 캐시 HIT
        RedisCache-->>HotTopicService: 캐시된 리스트 반환
        HotTopicService-->>Client: 리스트 반환
    else 캐시 MISS
        HotTopicService->>DB: 어제의 HotTopic 10개 조회
        HotTopicService->>RedisCache: 캐시 저장(만료: 내일 0시)
        HotTopicService-->>Client: 리스트 반환
    end

    Client->>HotTopicService: getNewsList(keyword, size)
    HotTopicService->>RedisCache: 캐시 조회(hottopic:news:{keyword})
    alt 캐시 HIT
        RedisCache-->>HotTopicService: 캐시된 뉴스 리스트 반환
        HotTopicService-->>Client: 리스트 반환
    else 캐시 MISS
        HotTopicService->>ES: 키워드 뉴스 검색
        HotTopicService->>RedisCache: 캐시 저장(만료: 내일 0시)
        HotTopicService-->>Client: 리스트 반환
    end

sequenceDiagram
    participant BatchJob
    participant DB

    BatchJob->>DB: 중복 뉴스 및 "[속보]" 포함 뉴스 삭제 쿼리 실행
    Note right of DB: published_at 범위 및 title 조건 동시 적용

sequenceDiagram
    participant User
    participant SettingController
    participant SettingService
    participant UserRepository

    User->>SettingController: saveSetting(settingDTO)
    SettingController->>SettingService: saveSetting(settingDTO)
    SettingService->>UserRepository: findById(userId)
    alt User 없음
        SettingService-->>SettingController: SettingException(USER_NOT_FOUND)
    else User 있음
        SettingService->>SettingService: isSettingLimitExceeded(user)
        alt 제한 초과
            SettingService-->>SettingController: SettingException(SETTING_LIMIT_EXCEEDED)
        else 제한 미초과
            SettingService->>SettingService: Setting 엔티티 생성 및 사용자 연결
            SettingService-->>SettingController: 완료
        end
    end

✨ Finishing Touches

📝 Generate Docstrings

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Explain this complex logic.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai explain this code block.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and explain its main purpose.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR.
@coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 2

🧹 Nitpick comments (6)

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Controller/SettingController.java (1)

75-75: TODO 항목에 대한 도움 제안

설정 개수 제한(3개) 검증 로직이 아직 구현되지 않았습니다. 서비스 레이어에서 관련 검증 메서드가 준비되어 있는 것으로 보입니다.

이 검증 로직을 구현하거나 새로운 이슈를 생성하여 추적하는 것을 도와드릴까요?
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (1)
246-253: 설정 제한 검증 로직 구현 필요

설정 개수 제한 검증을 위한 메서드가 스텁으로만 구현되어 있습니다. 실제 검증 로직이 필요합니다.

다음과 같은 구현을 제안드립니다:
private boolean isSettingLimitExceeded(User user) {
    long activeSettingsCount = settingRepository.countActiveSettingsByUser(user, LocalDateTime.now());
    return activeSettingsCount >= 3;
}
이 구현을 위해서는 SettingRepository에 countActiveSettingsByUser 메서드를 추가해야 합니다. 구현을 도와드릴까요?
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/HotTopic_Description (1)

7-7: 유용한 알림 코멘트입니다.

FIXME 코멘트를 통해 향후 개발에서 처리해야 할 요구사항을 명확히 문서화했습니다.

이 작업을 추적하기 위한 이슈를 생성하는 것을 고려해보시겠습니까? 도움이 필요하시면 말씀해 주세요.
monitoring/prometheus.yml (1)
25-25: 파일 끝에 개행 문자를 추가하세요.

YAML 파일 표준에 따라 파일 끝에 개행 문자가 필요합니다.
-      - targets: [ 'redis-session2-exporter:9121' ]
+      - targets: [ 'redis-session2-exporter:9121' ]
+
docker-compose.yml (1)
167-197: Redis 익스포터 구성이 우수합니다.

모든 Redis 인스턴스에 대한 모니터링 설정과 포트 충돌 방지가 잘 구성되어 있습니다.

선택적 개선사항: redis-session1-exporter와 redis-session2-exporter에도 해당 Redis 서비스에 대한 의존성을 추가하는 것을 고려해보세요:
  redis-session1-exporter:
    image: oliver006/redis_exporter
    container_name: redis-session1-exporter
    command:
      - '--redis.addr=redis://redis-session1:6379'
    ports:
      - "9122:9121"
+   depends_on:
+     - redis-session1
    networks:
      - backend
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java (1)
155-176: 뉴스 리스트 캐싱 구현이 일관성 있게 잘 구현되었습니다.

키워드 기반 캐시 키 전략과 성능 로깅이 핫토픽 캐싱과 일관되게 적용되었습니다.

선택적 개선사항: 키워드 기반 뉴스 검색 결과의 캐시 만료 전략을 핫토픽과 다르게 설정하는 것을 고려해보세요. 예를 들어, 뉴스 검색 결과는 더 짧은 TTL을 가질 수 있습니다:
- long ttlSeconds = Duration.between(
-         LocalDateTime.now(),
-         LocalDate.now().plusDays(1).atStartOfDay()
- ).getSeconds();
+ // 뉴스 검색 결과는 1시간 캐시
+ long ttlSeconds = Duration.ofHours(1).getSeconds();

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a25a67a and 06773e3.

📒 Files selected for processing (16)

.gitignore (1 hunks)
SpringBoot/build.gradle (1 hunks)
SpringBoot/database/init.sql (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/HotTopic_Description (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java (4 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Controller/SettingController.java (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (2 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Config/RedisConfig.java (2 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/ErrorCode.java (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/JWT/JwtAuthenticationFilter.java (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/Batch/listener/BatchJobCompletionListener.java (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/service/NewsEsService.java (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/NewsDeliverApplication.java (1 hunks)
SpringBoot/src/main/resources/application.properties (2 hunks)
docker-compose.yml (1 hunks)
monitoring/prometheus.yml (1 hunks)

🧰 Additional context used

🪛 YAMLlint (1.37.1)

monitoring/prometheus.yml

[error] 25-25: no new line character at the end of file

(new-line-at-end-of-file)

🔇 Additional comments (22)

SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/ErrorCode.java (1)

51-51: 새로운 에러 코드 추가 승인

설정 개수 제한 초과에 대한 새로운 에러 코드가 적절히 추가되었습니다. 명명 규칙을 잘 따르고 있으며, HTTP 상태 코드도 적절합니다.

.gitignore (1)

2-2: 좋은 변경사항입니다.

IntelliJ IDEA 프로젝트 파일을 .gitignore에 추가하는 것은 표준적인 관례입니다. 이를 통해 IDE 설정 파일이 저장소에 커밋되는 것을 방지할 수 있습니다.

SpringBoot/src/main/java/Baemin/News_Deliver/NewsDeliverApplication.java (2)

5-5: 적절한 import 추가입니다.

Spring의 캐싱 기능을 활성화하기 위한 올바른 import입니다.

12-12: 캐싱 기능 활성화가 올바르게 구현되었습니다.

@EnableCaching 어노테이션을 통해 Spring의 캐싱 추상화를 활성화했습니다. 이는 Redis 캐시 구현과 일치하며 올바른 위치에 배치되었습니다.

SpringBoot/src/main/resources/application.properties (3)

15-15: Redis 캐시 타입 설정이 올바르게 구성되었습니다.

Spring Boot의 캐시 추상화를 위한 적절한 Redis 설정입니다.

18-18: Redis 타임아웃 설정 검토가 필요합니다.

24시간의 Redis 타임아웃은 매우 긴 시간입니다. 이 설정이 애플리케이션의 요구사항과 일치하는지 확인해주세요.

다음 사항들을 검토해보세요:

캐시 데이터의 특성과 유효 기간

메모리 사용량에 대한 영향

데이터 일관성 요구사항

89-90: Prometheus 모니터링 설정이 적절하게 구성되었습니다.

health, info, prometheus 엔드포인트를 노출하고 애플리케이션별 메트릭 태그를 추가한 것은 좋은 모니터링 설정입니다.

SpringBoot/build.gradle (1)

85-91: 모니터링 및 캐싱 의존성 추가가 적절합니다.

Prometheus 모니터링과 Redis 캐싱 기능을 위한 의존성이 올바르게 추가되었습니다. Spring Boot의 의존성 관리를 활용하여 버전 충돌 없이 통합할 수 있습니다.

monitoring/prometheus.yml (1)

1-25: Prometheus 설정이 모니터링 목적에 적합합니다.

Spring Boot 애플리케이션과 여러 Redis 인스턴스에 대한 모니터링 설정이 올바르게 구성되었습니다. 5초 스크래핑 간격은 개발 환경에 적절합니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/ElasticSearch/service/NewsEsService.java (1)

97-101: SQL 쿼리 최적화가 성능 향상에 도움이 됩니다.

날짜 필터링 로직을 DATE() 함수 사용에서 타임스탬프 범위 조건으로 변경한 것은 좋은 개선입니다. 이 변경으로 인해 idx_news_published_sections 복합 인덱스를 효율적으로 활용할 수 있어 쿼리 성능이 향상됩니다.

SpringBoot/database/init.sql (1)

114-118: 데이터베이스 인덱스 최적화가 잘 설계되었습니다.

추가된 복합 인덱스들이 쿼리 성능 향상에 효과적입니다:

idx_news_published_sections: 날짜별 섹션 조회 성능 향상

idx_news_dup: 중복 삭제 로직의 ROW_NUMBER() 연산 최적화

각 인덱스의 컬럼 순서도 쿼리 패턴에 맞게 적절히 설정되었습니다.
SpringBoot/src/main/java/Baemin/News_Deliver/Global/News/Batch/listener/BatchJobCompletionListener.java (1)

41-53: 중복 삭제 로직 개선이 좋지만 속보 삭제 범위를 확인하세요.

타임스탬프 범위 필터링으로 변경한 것은 인덱스 활용 측면에서 우수합니다. 하지만 [속보] 삭제 조건이 날짜 필터링과 OR로 연결되어 있어 모든 날짜의 속보가 삭제될 수 있습니다.

의도한 동작이 맞는지 확인해주세요. 만약 어제 날짜의 속보만 삭제하려면 다음과 같이 수정을 고려해보세요:
-                WHERE (
-                    id IN (
-                        SELECT id FROM (
-                            SELECT id,
-                                   ROW_NUMBER() OVER (PARTITION BY title, publisher ORDER BY id) AS rn
-                            FROM news
-                            WHERE published_at >= CURDATE() - INTERVAL 1 DAY
-                              AND published_at < CURDATE()
-                        ) t
-                        WHERE t.rn > 1
-                    )
-                    OR title LIKE '%[속보]%'
-                );
+                WHERE id IN (
+                    SELECT id FROM (
+                        SELECT id,
+                               ROW_NUMBER() OVER (PARTITION BY title, publisher ORDER BY id) AS rn
+                        FROM news
+                        WHERE published_at >= CURDATE() - INTERVAL 1 DAY
+                          AND published_at < CURDATE()
+                          AND (title LIKE '%[속보]%' OR 1=1)
+                    ) t
+                    WHERE t.rn > 1 OR title LIKE '%[속보]%'
+                );
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Config/RedisConfig.java (4)

3-16: 필요한 의존성 임포트가 올바르게 추가되었습니다.

Jackson 직렬화 및 Redis JSON 시리얼라이저 관련 임포트가 캐시 기능 구현에 적절하게 추가되었습니다.

69-75: 캐시 전용 Redis 연결 팩토리가 올바르게 구성되었습니다.

세션용 Redis와 분리된 캐시 전용 연결 팩토리로 관심사 분리가 잘 되어 있습니다.

77-83: ObjectMapper 구성이 우수합니다.

Java 8 날짜/시간 타입 지원을 위한 JavaTimeModule 등록과 타임스탬프 비활성화가 올바르게 설정되어 있어 캐시에서 날짜 객체의 가독성이 향상됩니다.

85-103: 캐시 전용 RedisTemplate 구성이 우수합니다.

JSON 직렬화를 통한 복잡한 객체 캐싱 지원과 명확한 Bean 이름 지정이 잘 되어 있습니다. GenericJackson2JsonRedisSerializer는 타입 정보를 포함하여 안전한 역직렬화가 가능하지만, 저장 공간을 더 사용한다는 점을 참고하세요.

docker-compose.yml (3)

134-143: Prometheus 서비스 구성이 올바릅니다.

설정 파일 마운트와 영구 볼륨 구성이 적절하게 되어 있어 메트릭 데이터 지속성이 보장됩니다.

145-154: Grafana 서비스 구성이 적절합니다.

웹 인터페이스 포트 노출과 대시보드 데이터 지속성을 위한 볼륨 구성이 올바릅니다.

204-205: 모니터링 도구용 영구 볼륨이 올바르게 추가되었습니다.

Prometheus와 Grafana의 데이터 지속성을 위한 볼륨 구성이 적절합니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/service/HotTopicService.java (3)

51-59: 생성자 주입 방식이 올바르게 구현되었습니다.

특정 Redis 템플릿 빈 주입을 위한 @qualifier 사용으로 명시적인 생성자 주입이 적절하게 적용되었습니다.

72-105: 핫토픽 캐싱 구현이 우수합니다.

일일 데이터 특성에 맞는 캐시 키 전략, 자정까지의 TTL 계산, 그리고 성능 로깅이 잘 구현되어 있습니다. 특히 다음날 자정까지의 TTL 계산 로직이 일일 핫토픽의 특성에 매우 적합합니다.

61-61: 캐시 키 전략이 잘 정의되었습니다.

상수로 정의된 캐시 키 접두사를 통해 다양한 캐시 데이터 타입의 체계적인 관리가 가능합니다.

coderabbitai · 2025-07-18T02:42:08Z

+            User user = userRepository.findById(settingDTO.getUserId())
+                    .orElseThrow(() -> new SettingException(ErrorCode.USER_NOT_FOUND));
+
+            if (isSettingLimitExceeded(user)) {
+                throw new SettingException(ErrorCode.SETTING_LIMIT_EXCEEDED);
+            }
+
            Setting setting = new Setting();
            setting.setDeliveryTime(settingDTO.getDeliveryTime());
            setting.setStartDate(settingDTO.getStartDate());
            setting.setEndDate(settingDTO.getEndDate());
            setting.setIsDeleted(false);

-            User user = userRepository.findById(settingDTO.getUserId())
-                    .orElseThrow(() -> new SettingException(ErrorCode.USER_NOT_FOUND));
            setting.setUser(user);



⚠️ Potential issue

Setting 엔티티 저장 누락 문제

User 조회 및 검증 로직은 잘 구현되었지만, Setting 엔티티의 명시적 저장이 누락되었습니다. setting.getId()를 반환하려면 먼저 엔티티가 저장되어야 합니다.

다음과 같이 수정하여 Setting 엔티티를 저장하세요:

setting.setUser(user); + +settingRepository.save(setting); saveSettingKeyword(settingDTO, setting);

🤖 Prompt for AI Agents

In SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java around lines 64 to 78, the Setting entity is created and populated but not saved to the database, so calling setting.getId() will not work correctly. Fix this by explicitly saving the Setting entity using the appropriate repository's save method after setting its fields and before returning its ID.

coderabbitai · 2025-07-18T02:42:08Z

+        return path.startsWith("/login/oauth2/")
                || path.startsWith("/oauth2/")
-                || path.equals("/")
                || path.startsWith("/css/")
                || path.startsWith("/js/")
                || path.startsWith("/images/");


💡 Verification agent

🧩 Analysis chain

JWT 필터 제외 경로 변경을 검증해야 합니다.

/api/auth/ 및 / 경로를 제외 목록에서 제거했습니다. 이는 중요한 보안 변경사항으로, 해당 경로들이 이제 JWT 인증을 요구하게 됩니다.

/api/auth/ 경로가 로그인/회원가입 엔드포인트를 포함하는 경우, 이 변경으로 인해 접근이 불가능해질 수 있습니다

/ 경로 제외 제거로 메인 페이지 접근에 영향을 줄 수 있습니다

다음 스크립트를 실행하여 영향을 받는 엔드포인트들을 확인해주세요:

🏁 Script executed:

#!/bin/bash # /api/auth/ 경로를 사용하는 컨트롤러 확인 ast-grep --pattern $'@RequestMapping($$$) class $CLASS { $$$ }' | rg -A 10 -B 5 "api/auth" # 루트 경로 매핑 확인 ast-grep --pattern $'@RequestMapping($$$) $$$' | rg -A 5 -B 5 '^\s*"/"'

Length of output: 164

다음 스크립트로 /api/auth 및 루트 경로 매핑이 실제로 존재하는지 다시 확인해주세요:

🏁 Script executed:

#!/bin/bash # 1. /api/auth 경로 사용 여부 확인 grep -R "/api/auth" -n SpringBoot/src/main/java # 2. 메서드/클래스 레벨에서 루트("/") 매핑 확인 (@GetMapping, @PostMapping, @RequestMapping) grep -R -E '@(GetMapping|PostMapping|RequestMapping).*"\/"' -n SpringBoot/src/main/java # 3. path 속성 방식의 루트 매핑 확인 (@RequestMapping(path = "/")) grep -R 'path *= *"/"' -n SpringBoot/src/main/java

Length of output: 1015

/api/auth 경로 예외 처리 누락 확인 및 수정 필요

JwtAuthenticationFilter.java의 shouldNotFilter 메서드에 인증 관련 엔드포인트인 /api/auth/*가 제외되지 않아, 로그인·토큰발급·로그아웃 API 접근이 차단됩니다. 루트(/) 매핑 컨트롤러는 발견되지 않아 별도 영향은 없으나, 정적 리소스나 SPA 진입점이 있는 경우 예외 검토가 필요합니다.

위치:
SpringBoot/src/main/java/Baemin/News_Deliver/Global/JWT/JwtAuthenticationFilter.java (약 142행)

수정 제안: /api/auth/ 경로를 제외 목록에 추가 (path.startsWith("/api/auth/"))

예시 diff:

return path.startsWith("/login/oauth2/") || path.startsWith("/oauth2/") + || path.startsWith("/api/auth/") || path.startsWith("/css/") || path.startsWith("/js/") || path.startsWith("/images/");

필요 시 루트 경로 예외가 필요하다면 || path.equals("/")도 함께 추가하세요.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

return path.startsWith("/login/oauth2/")

|| path.startsWith("/oauth2/")

|| path.equals("/")

|| path.startsWith("/css/")

|| path.startsWith("/js/")

|| path.startsWith("/images/");

return path.startsWith("/login/oauth2/")

|| path.startsWith("/oauth2/")

|| path.startsWith("/api/auth/")

|| path.startsWith("/css/")

|| path.startsWith("/js/")

|| path.startsWith("/images/");

🤖 Prompt for AI Agents

In SpringBoot/src/main/java/Baemin/News_Deliver/Global/JWT/JwtAuthenticationFilter.java around lines 142 to 146, the shouldNotFilter method is missing exclusion for the /api/auth/ path, causing authentication-related endpoints to be blocked. Add a condition to exclude paths starting with "/api/auth/" by including `path.startsWith("/api/auth/")` in the return statement. Optionally, if root path exclusion is needed, also add `path.equals("/")`.

기존 try ~ catch 구조 제거 전역 핸들러에서 오류처리 유저의 Setting 개수 3개로 제한하는 로직 추가

coderabbitai

Actionable comments posted: 2

♻️ Duplicate comments (1)

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (1)
61-83: Setting 엔티티 저장 누락 문제 (중복 이슈)

설정 제한 로직 추가는 좋은 개선사항입니다. 하지만 이전 리뷰에서 지적된 Setting 엔티티 저장 누락 문제가 여전히 존재합니다. setting.getId()를 반환하려면 먼저 엔티티가 저장되어야 합니다.

다음과 같이 수정하세요:
 setting.setUser(user);
+
+settingRepository.save(setting);

 saveSettingKeyword(settingDTO, setting);

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d9a401f and ab9f096.

📒 Files selected for processing (4)

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/HotTopic_Description (1 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Controller/SettingController.java (3 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (4 hunks)
SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/GlobalExceptionHandler.java (2 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/HotTopic/HotTopic_Description
SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/Controller/SettingController.java

🔇 Additional comments (3)

SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/GlobalExceptionHandler.java (1)

46-54: 코드 스타일 개선 승인

SettingException 핸들러의 코드 포맷팅이 개선되어 가독성이 향상되었습니다.

SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java (2)

95-122: 메서드 시그니처 간소화 승인

ResponseEntity 반환 타입을 void로 변경하여 메서드가 간소화되었습니다. 권한 체크 로직과 로깅 메시지도 적절히 유지되고 개선되었습니다.

156-177: 소프트 삭제 로직 개선 승인

메서드 시그니처가 간소화되고 로깅 메시지가 개선되었습니다. 소프트 삭제 로직이 적절히 유지되어 데이터 무결성이 보장됩니다.

coderabbitai · 2025-07-18T03:37:36Z

+import org.springframework.web.bind.annotation.RestControllerAdvice;

 @ControllerAdvice
+@RestControllerAdvice


🛠️ Refactor suggestion

중복 어노테이션 사용 제거 필요

@ControllerAdvice와 @RestControllerAdvice를 동시에 사용하고 있습니다. @RestControllerAdvice는 @ControllerAdvice + @ResponseBody의 조합이므로 하나만 사용하면 됩니다.

다음과 같이 수정하세요:

-@ControllerAdvice -@RestControllerAdvice +@RestControllerAdvice

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

import org.springframework.web.bind.annotation.RestControllerAdvice;

@ControllerAdvice

@RestControllerAdvice

import org.springframework.web.bind.annotation.RestControllerAdvice;

@RestControllerAdvice

public class GlobalExceptionHandler {

// ...

}

🤖 Prompt for AI Agents

In SpringBoot/src/main/java/Baemin/News_Deliver/Global/Exception/GlobalExceptionHandler.java around lines 12 to 15, remove the redundant @ControllerAdvice annotation and keep only @RestControllerAdvice, since @RestControllerAdvice already includes @ControllerAdvice and @ResponseBody functionality.

coderabbitai · 2025-07-18T03:37:37Z

+    /**
+     * 사용자의 활성중인 설정이 3개가 넘는지 아닌지 검사하는 메서드 (내부 메서드)
+     * */
+    private boolean isSettingLimitExceeded(User user) {
+        List<Setting> settingList = settingRepository.findActiveSettings(user, LocalDateTime.now());
+
+        return settingList.size() > 3;
+    }


⚠️ Potential issue

설정 제한 로직 오류 수정 필요

메서드명은 isSettingLimitExceeded이지만 실제 로직은 > 3으로 되어 있어 4개까지 허용됩니다. 3개 제한을 의도했다면 >= 3 조건을 사용해야 합니다.

다음과 같이 수정하세요:

- return settingList.size() > 3; + return settingList.size() >= 3;

또는 메서드 문서화를 명확히 하세요:

/** - * 사용자의 활성중인 설정이 3개가 넘는지 아닌지 검사하는 메서드 (내부 메서드) + * 사용자의 활성중인 설정이 3개 이상인지 검사하는 메서드 (내부 메서드) * */

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

/**

* 사용자의 활성중인 설정이 3개가 넘는지 아닌지 검사하는 메서드 (내부 메서드)

* */

private boolean isSettingLimitExceeded(User user) {

List<Setting> settingList = settingRepository.findActiveSettings(user, LocalDateTime.now());

return settingList.size() > 3;

}

/**

* 사용자의 활성중인 설정이 3개 이상인지 검사하는 메서드 (내부 메서드)

* */

private boolean isSettingLimitExceeded(User user) {

List<Setting> settingList = settingRepository.findActiveSettings(user, LocalDateTime.now());

return settingList.size() >= 3;

}

🤖 Prompt for AI Agents

In SpringBoot/src/main/java/Baemin/News_Deliver/Domain/Mypage/service/SettingService.java around lines 221 to 228, the method isSettingLimitExceeded currently returns true only if the active settings count is greater than 3, allowing 4 active settings. To enforce a 3-setting limit as intended, change the condition to use >= 3 instead of > 3. Alternatively, update the method documentation to clearly state the current behavior if you want to keep the logic as is.

wjkim9 added 9 commits July 17, 2025 10:03

💡[주석 추가] : 추가해야 할 기능들 작성

771d3cc

코드 상의 FIXME 리스트 수행할 것.

✨[기능추가] : 설정 등록 중 검토 로직 추가

5d95476

로직 추가 구현중

Merge branch 'dev' into feat/hot-topic

d616f00

Merge branch 'dev' into feat/hot-topic

64357ad

🙈[.gitignore] : IntelliJ 설정 폴더 추가

265e5e6

IntelliJ 자동 구성 폴더 추가

📈[분석추가] : Prometheus 및 Grafana 모니터링 환경 구축

1821213

Prometheus를 통해 SpringBoot 서버와 Redis 서버 데이터 수집하여, Grafana로 대시보드 구현(시각화)

⚡️[성능 향상] : Redis-cache 서버를 활용하여 로드시간 개선

e17e94f

DB및 ElasticSearch 서버에 대한 접근 횟수를 줄이고 성능을 비약적으로 향상시킴

⚡️[성능 향상] : SQL 수정 및 INDEX 추가로 DB 성능향상

d21a1a4

중복 제거 로직에서 SQL을 INDEX를 적용받을 수 있도록 수정 중복 제거 시 복합 INDEX를 참조하여 성능이 향상 됨

Merge branch 'dev' into feat/hot-topic

06773e3

coderabbitai Bot requested changes Jul 18, 2025

View reviewed changes

wjkim9 added 2 commits July 18, 2025 11:59

Merge branch 'dev' into feat/hot-topic

d9a401f

♻️[리팩토링] : Setting 오류 처리 구조개선

ab9f096

기존 try ~ catch 구조 제거 전역 핸들러에서 오류처리 유저의 Setting 개수 3개로 제한하는 로직 추가

coderabbitai Bot requested changes Jul 18, 2025

View reviewed changes

Merge branch 'dev' into feat/hot-topic

7729b81

passionryu merged commit 728ec42 into dev Jul 18, 2025
1 check passed

coderabbitai Bot mentioned this pull request Jul 21, 2025

🐛 [버그 수정] : 스케줄러 및 메시지 샌딩 오류 수정 #116

Merged

Navigation Menu

Uh oh!

📈[분석추가] &⚡️[성능 향상] Prometheus & Grafana 모니터링 환경 구축 & 성능 향상 - #101

📈[분석추가] &⚡️[성능 향상] Prometheus & Grafana 모니터링 환경 구축 & 성능 향상#101
passionryu merged 12 commits into
devfrom
feat/hot-topic

wjkim9 commented Jul 18, 2025 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jul 18, 2025 •

edited

Loading

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Uh oh!

coderabbitai Bot Jul 18, 2025

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Uh oh!

coderabbitai Bot Jul 18, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

wjkim9 commented Jul 18, 2025 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Grafana로 Prometheus 데이터 시각화

Redis-cache 서버를 활용하여 로드 시간 개선

DB 쿼리 개선 및 인덱스 추가

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jul 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Chat

Support

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Jul 18, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

wjkim9 commented Jul 18, 2025 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jul 18, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)