Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BE] TooManyRequest 요청을 로깅할 때, LB 에서 전달되는 X-Forwarded-For 헤더를 추출하여 공인 IP를 기록한다. #943

Closed
nayonsoso opened this issue Oct 24, 2024 · 1 comment · May be fixed by #944
Closed

[BE] TooManyRequest 요청을 로깅할 때, LB 에서 전달되는 X-Forwarded-For 헤더를 추출하여 공인 IP를 기록한다. #943

nayonsoso opened this issue Oct 24, 2024 · 1 comment · May be fixed by #944
Assignees
@nayonsoso
Labels
✨ BE 🍰 초간단
Milestone
6차 스프린트: 최종장

Comments

@nayonsoso
Copy link
Contributor

🔍 설명

  • 현재는 로드벨런서의 IP 를 remoteAddress 로 로깅하고 있습니다.
  • 이는 의미가 없으며.. LB 가 서버에 전달할 때 전해주는 X-Forwarded-For 헤더에서 진짜 IP 를 알 수 있습니다.
  • 이를 위한 리팩터링을 진행합니다.

🔥 할 일

  • RequestLimitInterceptor 의 코드를 수정합니다.

⏰ 예상 시간

  • 15분
@nayonsoso nayonsoso self-assigned this Oct 24, 2024
@nayonsoso nayonsoso linked a pull request Oct 25, 2024 that will close this issue
[BE] refactor: TooManyRequest 로깅에 사용되는 IP 추출 로직 변경 #944
Draft
@nayonsoso nayonsoso mentioned this issue Oct 25, 2024
Draft
@donghoony
Copy link
Contributor

Nginx에서 해당 내용을 처리하기로 결정함에 따라 해당 이슈 close할게요~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nayonsoso nayonsoso

Labels
✨ BE 🍰 초간단
Projects
2024-review-me
Status: Done
Milestone
6차 스프린트: 최종장
2 participants
@donghoony @nayonsoso