Skip to content

fix: 조회수 IP 중복방지 Redis 전환 (HPA pod 간 공유)#413

Merged
sdk-kr merged 3 commits into
mainfrom
fix/viewcount-redis-dedup
Mar 8, 2026
Merged

fix: 조회수 IP 중복방지 Redis 전환 (HPA pod 간 공유)#413
sdk-kr merged 3 commits into
mainfrom
fix/viewcount-redis-dedup

Conversation

@sdk-kr
Copy link
Copy Markdown
Contributor

@sdk-kr sdk-kr commented Mar 8, 2026

Summary

  • 조회수 IP 기반 중복방지를 인메모리 Map에서 Redis SETEX로 전환
  • K8s HPA 환경(4-12 pod)에서 새로고침 시 다른 pod 라우팅으로 조회수가 무한 증가하던 버그 수정
  • Redis 키: vc:{ip}:{boardId}:{postId}, TTL 10분
  • Redis 장애 시 graceful fallback (조회수 약간 늘어날 수 있으나 크래시 없음)

Test plan

  • pnpm build 성공
  • dev.damoang.net 게시글 페이지 200 정상 응답
  • 운영 배포 후 조회수 증가 패턴 모니터링

Deploy Bot and others added 3 commits March 8, 2026 18:15
카나리 배포+스모크 테스트 통과 후 텔레그램으로
커밋 메시지 + GitHub Actions 승인 링크를 전송.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 전역 preload-data를 hover에서 tap으로 변경하여 __data.json 과다 요청 방지
- 18개 컴포넌트의 개별 hover 속성 제거 (전역 tap 상속)
- 홈페이지 celebration 중복 로드 제거 (layout에서 이미 로드)
4-12개 K8s pod이 각각 독립적인 인메모리 Map을 가지고 있어
새로고침 시 다른 pod으로 라우팅되면 조회수가 중복 증가하던 버그 수정.
Redis SETEX (10분 TTL, vc: 접두사)로 전환하여 pod 간 상태 공유.
@sdk-kr sdk-kr merged commit fcc85a4 into main Mar 8, 2026
7 of 11 checks passed
@sdk-kr sdk-kr deleted the fix/viewcount-redis-dedup branch March 8, 2026 09:42
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.

1 participant