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.

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

로그 수집 내용 추가 #430

Open
kong-hana01 opened this issue Nov 9, 2023 · 1 comment
Open

로그 수집 내용 추가 #430

kong-hana01 opened this issue Nov 9, 2023 · 1 comment
Assignees
Labels
backend 백엔드 분야

Comments

@kong-hana01
Copy link
Collaborator

기능 설명

현재 로그 수집 정보는 다음과 같습니다.

  • Request URI
  • Request HTTP Method
  • Response HTTP Status
  • 소요 시간

최근 다른 팀들의 aws를 보니 외부에서 이상한 요청이 계속 몰려오는 것 같아요.
일단 로그 수집 정보를 늘려보는건 어떨까요?

그리고 추가적으로 예전에 로그 수집 목적에 '사용자에 관한 cs 관리'에 대한 내용도 있었는데
현재 사용자에 대해 수집한 정보가 아예 없어서 사용자가 전화하거나 문제가 생겼을 때 저희에게 요청을 하더라도,
올바르게 사용자를 식별하는 과정이 어렵다고 느껴집니다!

제안하고자하는 로그 수집 정보는 다음과 같습니다!!

  • 요청 ip address
  • 요청한 사람의 memberId(세션 정보를 활용해 memberId를 추출해야할 것 같아요.)

한번 읽어보시고 얘기해주세용

개발 일정

No response

자식 이슈

No response

부가 설명

No response

@Songusika
Copy link
Collaborator

" 요청 IP 주소는 저희가 앞단에 nginx 를 두고 있으니 nginx 에서 하는건 어떨까요? "
라고 적고 있었는데 X-Forwarded-For 헤더에 클라이언트 IP 주소가 담기겠군요 ㅎㅎ

추가적으로 memberId 는 로그인하지 않은 유저에 대한 처리까지 같이 하면 좋을 것 같습니다!
아래와 같이 LogFilterMemberArugmentResolver 두 곳에서 쿠키에 담긴 값을 가져와야하는데
memberId 키 값을 문자열로 두 곳에서 관리하는게 조금 깨림직하네요 이걸 해결할 수 있는 방법이 있을까요??

(Long) webRequest.getAttribute("memberId", WebRequest.SCOPE_SESSION);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 분야
Projects
Status: Todo
Development

No branches or pull requests

4 participants