Skip to content

Conversation

@doup2001
Copy link
Member

@doup2001 doup2001 commented Jan 20, 2026

📌 작업한 내용

RedisConfig 설정 문제를 수정하여 Redis 연결 및 캐싱 기능이 정상 동작하도록 했습니다. 이전 시도에서 발생한 연결 오류를 해결하고, 설정을 최적화했습니다.

🔍 참고 사항

  • Redis 연결: 호스트, 포트, 인증 정보 재설정 및 환경변수 연동
  • 캐싱 설정: TTL 및 직렬화 방식 최적화
  • 에러 핸들링: 연결 실패 시 graceful degradation 처리 추가
  • 테스트: 로컬 및 스테이징 환경에서 Redis 연동 확인

🖼️ 스크린샷

UI 변경 사항 없음

🔗 관련 이슈

Redis 설정 관련 이슈 (2차 수정)

✅ 체크리스트

  • Redis 연결 테스트 완료
  • 캐싱 동작 검증
  • 환경별 설정 차이 적용
  • 문서화 업데이트

Summary by CodeRabbit

  • 버그 수정
    • Redis 데이터베이스 연결 설정이 개선되어 프로덕션 환경에서 SSL/TLS 보안이 자동으로 활성화됩니다. 이를 통해 데이터 전송 보안이 더욱 강화되었습니다.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Caution

Review failed

The pull request is closed.

개요

RedisConfig의 redisConnectionFactory 메서드가 Environment 파라미터를 받도록 수정되었습니다. 환경 프로파일을 확인하여 SSL 활성화 여부를 결정하는 로직이 추가되었으며, prod 프로파일 또는 설정된 sslEnabled 플래그에 따라 SSL을 조건부로 활성화합니다.

변경 사항

Cohort / File(s) 변경 내용
Redis 설정 업데이트
src/main/java/com/pinHouse/server/core/config/RedisConfig.java
redisConnectionFactory 메서드 시그니처를 수정하여 Environment 파라미터 추가. EnvironmentProfiles import 추가. useSsl 부울 변수 도입으로 sslEnabled 플래그와 prod 프로파일 여부를 모두 확인. SSL 활성화 조건을 새로운 useSsl 변수로 업데이트.

예상 코드 리뷰 소요 시간

🎯 2 (단순) | ⏱️ ~8분

🐰 Redis의 SSL을 점검하네,
환경을 재빨리 감지하고,
Prod 환경이면 암호화 활성화!
설정의 마법으로 보안 강화,
작은 변화가 큰 보호를 만든다네~

✨ Finishing touches
  • 📝 Generate docstrings

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.

@github-actions
Copy link

Test Results

22 tests  ±0   22 ✅ ±0   0s ⏱️ ±0s
 3 suites ±0    0 💤 ±0 
 3 files   ±0    0 ❌ ±0 

Results for commit 2317303. ± Comparison against base commit 3bbd134.

@doup2001 doup2001 merged commit b129b6a into main Jan 20, 2026
2 of 3 checks passed
@doup2001 doup2001 deleted the hotfix/리다이렉트 branch January 20, 2026 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants