Skip to content

Conversation

@Jaehyeon-Han
Copy link
Collaborator

@Jaehyeon-Han Jaehyeon-Han commented Nov 18, 2025

변경 사항

OpenResty와 Redis를 통한 rate limiting 구현

  • Redis Functions를 이용한 token bucket 알고리즘 구현
  • OpenResty rate limiting 설정

로컬 도커 환경에서의 Redis Sentinel 구축

  • 3개의 컨테이너가 각 Redis 및 Sentinel로 동작하도록 구성

리뷰 요청 사항과 질문

  • 로컬 도컬 환경에서 Sentinel 구축 경험을 의미 있는 수준으로 올리려면?
  • header에 user_id를 넣는 것은 좋지 않은가? JWT라면 프록시가 파싱해도 되는가?
  • 프록시에서 rate limiting을 하는 것보다 어플리케이션에서 rate limiting하는 게 낫나?
  • 여러 nginx 노드를 사용할 때, 통일해야 하는 설정값은 어떻게 관리? (e.g. conf 파일, 환경 변수)

보완 계획

  • JWT 기반으로 인증 방식 통일, app 요청 DTO에서 userId 제거 후 헤더로 변경
  • Redis fail over 또는 지연 시 Nginx 로직 개선
  • app과 nginx 프록시 연결
  • (rate limiting 테스트 코드 작성)

closes #7

@Jaehyeon-Han Jaehyeon-Han self-assigned this Nov 18, 2025
@sonarqubecloud
Copy link

@Jaehyeon-Han Jaehyeon-Han marked this pull request as ready for review November 19, 2025 06:02
@Jaehyeon-Han Jaehyeon-Han force-pushed the main branch 4 times, most recently from b2cb863 to f41899d Compare November 19, 2025 07:23
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.

사용자별 채팅 전송에 Rate Limiting 적용

2 participants